Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: patch형태의 update API에서 falsy값인 경우 notify가 되지 않는 버그 수정 #316

Merged
merged 5 commits into from
Jul 21, 2024

Conversation

choyoungwoo9
Copy link
Collaborator

✅ 작업 내용

  • update후 notify할 프로퍼티의 여부를 falsy로 체크하는게 아니라 undefined인지 체크하게 변경
  • task를 업데이트 할 때 프로퍼티의 값이 falsy값인 경우 테스트 추가

- epic 수정 API에 적용
- story 수정 API에 적용
- task 수정 API에 적용
- member 수정 API에 적용
- 상위 테이블 삭제시 유효하지 않은 정보는 함께 삭제되게 구현
- 상위 테이블 삭제에도 유효한 정보(메모-멤버, 태스크-멤버)는 NULL로 설정
@choyoungwoo9 choyoungwoo9 requested a review from kimsj-git July 21, 2024 13:01
@choyoungwoo9 choyoungwoo9 merged commit b8208b9 into dev Jul 21, 2024
1 check passed
@choyoungwoo9 choyoungwoo9 deleted the feature/update-notify-api-fix branch July 21, 2024 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant