Merged
Conversation
- 읽은 공지 여부 조회 로직을 Left Join으로 1회 쿼리로 불러오도록 개선 진행 - 공지 상세 조회시 읽기로 변환하는 로직을 Select -> Insert가 아닌 Upsert로 저장하도록 수정
- notification_id, user_id를 유니크 제약 조건으로 명시함
- 기존 GET /notifications/unread 는 API명은 명확하지만 Restful하지 않다고 판단 - 확장성을 고려하여 GET /notifications/status 를 추가 작성 (알림 내역 상태이기 때문에 읽지 않은 알림 개수 등 확장이 가능)
[REFACTOR] 패키지 분리 및 Restful API 리팩토링
[REFACTOR] Notification 리팩토링 건 및 키워드, 매력포인트 추가 건 개발 서버 배포
[FIX] 피드 조회시 distinct 적용
[FEAT] 피드 작성 시간 표기 정책 변경 (Relative Time 포맷 적용)
[FIX] iOS 애플 로그인 유저 회원 탈퇴 실패 (Client Secret 생성 오류)
[FIX] @positive를 @PositiveOrZero로 수정
- CreatedAt, UpdatedAt 필드가 존재하지 않는데, Native 쿼리에서 해당 필드를 저장하려고 해서 발생하는 문제
[FIX] NativeQuery 오작성으로 인한 SQL 호출시 오류가 발생하는 문제 수정
[FIX] 피드에서 피드 카테고리 제거
[FIX] 소설 오늘의 발견 조회시 비공개, 스포일러 피드 제외하고 조회
[FIX] 인기글 조회시 선호장르 없으면 전체 장르 조회
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related Issue
Key Changes
To Reviewers
References