-
Notifications
You must be signed in to change notification settings - Fork 2
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
[OING-379] feat: 알림 목록 조회 기능 추가 #280
base: dev
Are you sure you want to change the base?
Conversation
… and MissionUnlocked
Test Results 54 files 54 suites 11s ⏱️ Results for commit 1317c09. ♻️ This comment has been updated with latest results. |
Code Coverage
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
추가 논의되어야 하는 부분들이 많아 스타일적인 거에 대해서만 코멘트 남겼습니다! 복잡한 기능이라 구현할 부분이 많았는데 수고하셨어요~~
gateway/src/main/java/com/oing/domain/UserNotificationHistory.java
Outdated
Show resolved
Hide resolved
gateway/src/main/java/com/oing/dto/response/NotificationResponse.java
Outdated
Show resolved
Hide resolved
… feature/OING-379
Quality Gate passedIssues Measures |
❓ 기능 추가 배경
�알림 이력 화면에서 최근 한 달간 발생한 알림을 조회하기 위해, 관련 서버 기능을 추가했습니다.
➕ 추가/변경된 기능
🥺 리뷰어에게 하고싶은 말
일여님께 대부분 답변 받아, 관련 부분들 반영 완료했습니다.
알림 이력 기능에 대해 기획적으로 논의해야할 부분이 있어, 우선 필수적인 코드들을 우선 작성하고 PR을 올립니다.전체 회의에서 추가 논의되어야 할 부분들은 다음과 같습니다.
푸시알림과 알림이력은 서로다른 개념인가요? 직관적으로 봤을때는 동일한 개념으로 보이지만, 푸시알림에만 존재하는 알림 종류가 있고 알림이력에만 존재하는 알림 종류가 있기 때문에, 한 번 짚고 넘어가는 것이 좋을 것 같습니다.메인화면으로 이동하는 것과 신버전 출시시 업데이트 페이지로 가기위한 딥링크가 존재할까요? 그리고 프론트 입장에서 딥링크 주소로 null이 입력될경우, 앱이 어떻게 동작할까요? 특정 알림의 경우 이 부분에 대한 명시가 되어있지 않아, 확인이 필요합니다.생일 안내 알림의 경우, 몇 시에 발송되는 것이 좋을까요?-> 당장 방법이 없고, API 로 열어두기에는 프로덕션에 영향도가 너무 커서 보류합니다.
🔗 참조 or 관련된 이슈
https://no5ing.atlassian.net/browse/OING-379
🔗 참조 or 관련된 이슈
https://no5ing.atlassian.net/browse/OING-379