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

Api: ✏️ 확인 알림과 미확인 알림 조회 API 분리 #161

Merged
merged 11 commits into from
Oct 8, 2024

Conversation

psychology50
Copy link
Member

작업 이유

  • 기존에 확인 알림과 미확인 알림을 합쳐버리니, 무한 스크롤이 뒤섞이는 문제가 발생함.
  • iOS 측 요청에 따라 확인 알림과 미확인 알림을 분리

작업 사항

  • 기존 API 변경 사항
    • 미확인 알림 존재 여부를 판단하기 위한 URL이 GET /notifications/unreadGET /notifications/unread/exist로 수정
  • 구현된 API
    • url: GET /notification/unread
    • 해당 경로로 요청한 데이터는 무한 스크롤이 아닌, 전체 데이터를 모두 반환함. (단, 모든 데이터는 시간 순(id)으로 역정렬되어, 최신순서가 가장 앞의 인덱스에 위치.)

리뷰어가 중점적으로 확인해야 하는 부분

  • 미확인 알림 여부 조회를 위한 경로를 /notifications/unread/exist로 일단 지정해뒀는데, exist 대신 대체할 만한 게 있을 지..?

발견한 이슈

  • 현재 푸시 알림 타입 구분없이 모든 알림에 대해 무한 스크롤을 사용하는데, 채팅 메시지도 여기서 확인 가능하게 되면 문제가 생길 여지가 있음. 이에 대해 디자이너와 상의가 필요함.

@psychology50 psychology50 added the fix 기능 수정 label Oct 8, 2024
@psychology50 psychology50 self-assigned this Oct 8, 2024
@psychology50 psychology50 merged commit d1ae456 into dev Oct 8, 2024
1 check passed
@psychology50 psychology50 deleted the fix/PW-561-divide-get-push-notification-api branch October 8, 2024 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 기능 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant