Skip to content

Conversation

@rinarina0429
Copy link
Member

Related Issue

Key Changes

알림 단건 조회 API를 구현했습니다.

  • 알림이 공지 타입(현재는 공지/이벤트가 해당)인지 확인하고
  • 알림 대상이 전체(0) 혹은 유저 자신인지 확인하고
  • 만약 해당 유저와 알림에 대한 ReadNotification이 없다면 ReadNotification 추가
  • 마지막으로 공지 단건 내용 리턴
    방식으로 이루어집니다.

To Reviewers

  • 제가 알기로는 공지에 맞는 타입인 경우에만 공지 상세조회가 되고, 아닌 경우에는 피드로 이동한다고 알고 있어서 공지 타입이 아닌 경우 아예 Exception을 터지게 했는데요, 혹시 공지 타입이 아니더라도 상세 조회가 가능하게 하는게 좋을까요?

References

@rinarina0429 rinarina0429 self-assigned this Jan 16, 2025
@github-actions github-actions bot requested review from ChaeAg and Kim-TaeUk January 16, 2025 13:54
Copy link
Member

@ChaeAg ChaeAg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다!! 빠른 구현 굿굿 :>
PR 본문에서 말씀하신 건 저는 예외를 발생시켜야 한다고 생각해요. 푸시알림까지 상세조회가 되버리면 안되는 거니까요!
코멘트 확인 바라요~~

@ChaeAg ChaeAg removed the request for review from Kim-TaeUk January 17, 2025 10:40
@ChaeAg ChaeAg merged commit 36dd89b into dev Jan 17, 2025
1 check passed
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.

[FEAT] Notification 단건 조회 API 구현

3 participants