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

FIX : 이전 버전 알림 API 재생성 및 Deprecated 처리 #149

Merged
merged 2 commits into from
Nov 19, 2023

Conversation

yummygyudon
Copy link
Member

  • (신버전) 알림 상세 조회 URI : /detail/{notificationId} 로 변경
  • (신버전) 알림 리스트 조회 URI : /all 로 변경
  • (신버전) 알림 읽음 처리 URI : /read (전체), /read/{notificationId} (개별) 로 변경

📝 PR Summary

🌵 Working Branch

feat/#148-create-legacy-alarm-api

🌴 Works

  • 구버전 API 재생성 및 Deprecated 처리
    • Response DTO Deprecated 처리
    • Notification Entity ID 기반 Service Method Deprecated 처리
    • Notification Entity ID 기반 Controller Method Deprecated 처리
  • 신버전 API URI 분리
    • (신버전) 알림 상세 조회 URI : /detail/{notificationId} 로 변경
    • (신버전) 알림 리스트 조회 URI : /all 로 변경
    • (신버전) 알림 읽음 처리 URI : /read (전체), /read/{notificationId} (개별) 로 변경

🌱 Related Issue

#148

- (신버전) 알림 상세 조회 URI : `/detail/{notificationId}` 로 변경
- (신버전) 알림 리스트 조회 URI : `/all` 로 변경
- (신버전) 알림 읽음 처리 URI : `/read` (전체), `/read/{notificationId}` (개별) 로 변경
@yummygyudon yummygyudon requested a review from gunom November 18, 2023 15:16
@yummygyudon yummygyudon self-assigned this Nov 18, 2023
@yummygyudon yummygyudon linked an issue Nov 18, 2023 that may be closed by this pull request
5 tasks
@yummygyudon
Copy link
Member Author

최대한 Stamp 도메인 참고해서 작성해봤는데 놓친 부분있는지 확인부탁드림다!!
예전 방식처럼 Entity ID 기반으로 테스트 해봤는데 성공했구
리스트의 경우도 이전 버전으로 하면 Long인 Entity ID가 들어있는 DTO 로 내려가구
신버전으로 하면 String 인 Unique ID가 들어있는 DTO로 내려가는 것을 확인했습니담

Copy link
Contributor

@gunom gunom left a comment

Choose a reason for hiding this comment

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

좋습니다! 구버전 지원되는지 테스트는 아직이죠?

@yummygyudon
Copy link
Member Author

좋습니다! 구버전 지원되는지 테스트는 아직이죠?

@gunom 로컬에서는 테스트 완료했습니당!! 슬랙 DM으로 보내놓긴했어요!!

@yummygyudon yummygyudon merged commit c99564c into dev Nov 19, 2023
2 checks passed
@ozzing ozzing deleted the feat/#148-create-legacy-alarm-api branch May 23, 2024 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FEAT : 알림 관련 구버전/신버전 API 분리
2 participants