Skip to content

Conversation

@rinarina0429
Copy link
Member

Related Issue

Key Changes

홈 화면에서 알림을 안읽었을 시 나타나는 빨간 점의 여부를 조회하는 기능을 구현했습니다.
Notion API 명세서

To Reviewers

  • 알림 단건 조회와 커밋이 겹쳐 볼게 많아보인다면, 단건조회 pr 후 리뷰해주셔도 좋습니다! 이 pr 자체는 변경사항이 많지는 않아서요
  • 현재 UIR가 /notifications/unread인데, 뭔가 딱 좋다! 느낌이 아니라서, 한번씩 확인 부탁드립니다! 바꿀만한게 있으면 남겨주세요!

References

@rinarina0429 rinarina0429 self-assigned this Jan 17, 2025
@github-actions github-actions bot requested review from ChaeAg and Kim-TaeUk January 17, 2025 03:28
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.

굿굿입니다! :>
코멘트 하나 남겼으니 확인바래요~!~!
엔드포인트는 흠 boolean값만을 응답하니까 /has-unread, /unread-check정도 생각나는데 기가막힌 아이디어는 없는 것 같아유 🥲
+현재 API 이름이 홈 화면 빨간점 여부 조회로 작성되어 있는데, 유저 비열람 알림 존재 여부 조회 처럼 기능의 목적을 명확하게 반영하는 이름이 더 적절할 것 같습니다.

@ChaeAg ChaeAg removed the request for review from Kim-TaeUk January 17, 2025 10:55
@ChaeAg ChaeAg merged commit a62e5cb into dev Jan 17, 2025
2 checks 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] 홈화면 알림 빨간점 여부 조회 기능 구현

3 participants