Skip to content

Conversation

@earthyoung
Copy link
Contributor

What is this PR?

투데이에서 투두 모달에서 '인박스에 넣기'를 눌러도 투두가 인박스로 들어가지 않는 문제를 수정했습니다.

Changes

원인은 투두의 Date를 null로 설정할 때 categoryId 값도 null 이었기 때문이었는데, 알고보니 useContext()를 통해 현재는 사용하지 않는 selectedCategory 라는 변수를 카테고리 값으로 참고하고 있었기 때문이었습니다.
이에 selectedCategory를 참조하는 대신, categoryId를 직접 prop으로 전달해서 문제를 해결했습니다.

Screenshots

Test Checklist

😎github action에서 CI / 코드 품질 검사 (pull_request) 실패하면 details 눌러서 수정할 부분 보고 고치기


😎앱 테스트


안드로이드 👉 APP_MODE=development npm run e2e-test:android


iOS 👉 손으로 테스트 ㅠㅠ

기능체크리스트

  • 투두 CRUD
  • 서브투두 CRUD
  • 서브투두 AI 생성
  • 투두 날짜 옮기기
  • 투두 인박스로 이동
  • 인박스에서 위 3과정 똑같이 진행
  • 카테고리 CRUD
  • 캘린더에 남은 할일 수 확인
  • 설정에 계정, 언어, 문의 페이지 확인

@byungchanKo99
Copy link
Collaborator

잘 동작하네요!

@byungchanKo99 byungchanKo99 merged commit 9ec720f into dev Nov 22, 2024
2 of 3 checks passed
@byungchanKo99 byungchanKo99 deleted the SZ-551 branch November 22, 2024 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants