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

CLAP-411 미확인 알림 개수 조회 오류 해결 #535

Merged
merged 3 commits into from
Feb 13, 2025
Merged

Conversation

starboxxxx
Copy link
Collaborator

📄 요약(Summary)

미확인 알림 개수 조회 오류 해결

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • 오래된 데이터가 DB에 남아있어 사용자가 취소한 작업에 대한 알림이 지워지지 않고 남아있었고 이로 인해 미확인 알림개수 조회에서 오류가 발생하였습니다.
  • 이 부분을 해결하기 위해 별도로 count Query에 isDeleted=false인 데이터만 조회하도록 조건을 추가해주었습니다.

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • [ ✅ ] PR 양식에 맞게 작성했습니다
  • [ ✅ ] 모든 테스트가 통과했습니다
  • [ ✅ ] 프로그램이 정상적으로 작동합니다
  • [ ✅ ] 적절한 PR 라벨을 설정했습니다
  • [ ✅ ] 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #533

@starboxxxx starboxxxx linked an issue Feb 12, 2025 that may be closed by this pull request
2 tasks
@starboxxxx starboxxxx self-assigned this Feb 12, 2025
@starboxxxx starboxxxx added the 🐛 bug 버그를 수정합니다 label Feb 12, 2025
@joowojr joowojr merged commit c4f773d into develop Feb 13, 2025
1 check passed
@joowojr joowojr deleted the CLAP-411 branch February 13, 2025 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug 버그를 수정합니다
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CLAP-411 미확인 알림 개수 조회 에러
2 participants