Skip to content

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 미확인 알림 개수 조회 에러

3 participants