fix : 피드 조회 #259
Merged
fix : 피드 조회 #259
Conversation
This file contains 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
|
Lint 체크만 한번 부탁드립니다! |
binary-ho
approved these changes
Apr 24, 2024
egg528
added a commit
that referenced
this pull request
Apr 26, 2024
* feat: remove forward header optiion * refactor : birth 관련 유저 정보 업데이트 API에 birth를 nullable로 변경 (#206) (#207) * feat(#209): spring security authorization_request_not_found 수정 (#210) * [🌎 Feature] Goal 댓글 API 개발 (#208) * feat : 댓글 엔티티와 repository 생성 (#196) * feat : Goal 댓글 오름차순 전체 조회 메서드 구현 (#196) * feat : Goal 댓글 작성 삭제 메서드 구현 (#196) * feat : Goal의 마지막 댓글 Read 시간을 저장하는 GoalCommentReadTime 구현 (#196) * feat : Goal의 생성자가 읽지 않은 댓글이 있는지 확인하는 메서드 구현 (#196) * feat : 비동기적으로 마지막 댓글 확인 시간을 update하는 메서드 구현 (#196) * ktlint formatting * refactor : 마지막 댓글 Read 시간을 분리된 테이블이 아닌 Goal이 갖도록 변경 (#196) * feat : 댓글 조회시 요청자가 Goal의 주인이라면 comment read time을 update하는 이벤트 발행 (#196) * test : Goal 필드 추가에 따른 테스트 코드 변경 (#196) * refactor : CoreApiException이 직접 errorMessage를 입력 받을 수 있도록 변경 (#196) * refactor : Comment가 스스로 제한 조건을 검증하도록 변경 (#196) * refactor : CommentService의 필드들에 private 접근 제한자 추가 (#196) * refactor : comment require 조건 정정 (#196) * refactor : 불필요한 트랜잭션 전파 및 Transactional 이벤트 리스너 제거 (#196) * refactor : 불필요한 Goal 조회 쿼리 제거 (#196) * test : Comment 관련 기능 통합 테스트 코드 작성 (#196) * ktlint formatting * refactor : getAllByGoalId 메서드의 이름을 역할에 맞게 findAllByGoalId로 변경 (#196) * feat : 댓글 API 구현 (#196) * chore : 잘못 작성된 메서드 이름 변경 (#196) * refactor : DTO 패키지 변경 (#196) * refactor : comments에서 goal을 꺼낼 때, first 메서드 활용 (#196) * refactor : 불필요한 @async 어노테이션 제거 (#196) * refactor : findAllByGoalId 가독성 개선 (#196) * refactor : Emoji API Restful하게 개선 (#196) (#215) * [🚌 Issue ] Emoji Count 작업 및 Count 관련 로직 개선 (#213) * refactor(#211): count관련 eventListener 단일 component화 * feat(#211): emoji count 테이블 추가 및 count관련 event application lock 적용 * feat(#211): add goal count event에 exclusive runner 적용 * refactor: event 같은 패키지로 이동 * feat: lock 적용 시점 비동기 이벤트 실행 Thread로 변경 * refactor: event 네이밍 컨벤션 적용 * fix: key에 매핑된 lock 대기 thread가 없다면 map에서 제거 * [🚌 Issue] 리팩토링 이슈 템플릿 세팅 (#220) * chore : 리팩토링 이슈 템플릿 세팅 * chore : 기타-이슈 기본 생성 제목에 불필요한 띄어쓰기 제거 * [🛠️ Refactor] Goal Emoji API 응답에 "이모지를 누른 사람 수" 추가 (#221) * refactor : Goal에 반응 조회 API에 반응한 사람 수 값 추가 (#218) * test : Goal 반응 조회 API 반응한 사람 수 값이 정확한지 확인하는 테스트 작성 (#218) * [🛠️ Refactor] IsMyReaction을 확인할 때 Username이 아닌 id로 확인하기 (#219) * refactor : IsMyReaction을 확인할 때 Username이 아닌 id로 확인하도록 변경 (#217) * merge with develop * [🆘 Bug] authorization_request_not_found 오류 (#216) * feat(#209): spring security authorization_request_not_found 수정 * fix: AuthorizationRequestRepository cookie 기반에서 state parameter 기반으로 변경 * fix: AuthorizationRequestRepository remove 메서드에 명시적인 삭제 추가 * fix: AuthorizationRequestRepository oauthRequestStorage 데이터 명시적 삭제 로직 제거 * refact(#171): add emoji count * feat(#171): 피드에 이모지및 내가 누른 이모지 추가 * chore(#171): ktlint * chore(#171): 사용하지 않는 코드 삭제 * chore(#171): ktlint * [🛠️ Refactor] 댓글 응답에 댓글 갯수 추가 (#225) * refactor : Goal에 달린 댓글 응답에 댓글 갯수 추가 (#224) * test : Goal에 달린 댓글 응답이 댓글 갯수를 반환하는 것을 확인하는 테스트 코드 작성 (#224) * ktlint formatting * refact(#171): edit field name * [🛠️ Refactor] RunnerLock 필드 변수명 변경 �+ Security Uri 상수화 + H2 웹 콘솔용 WebSecurityCustomizer 제거 (#226) * refactor: RunnerLock 내부 필드 변수명 변경 * refactor: security permitall uri 상수화 * [🛠️ Refactor] 댓글 조회 API 응답에 "내 지도 확인" key-value 추가 (#230) * refactor : CommentsResponse에 자신의 목표인지 확인하는 isMyGoal 필드 추가 (#229) * test : CommentsResponse를 통해 자신의 목표인지 확인할 수 있는지 검증하는 테스트 코드 작성 (#229) * feat(#232): add title field * [🛠️ Refactor] 프로젝트 구조 변경 (1번, 3번 제안만 변경) (#234) * refactor(#235): pagination 공통화 + cheering squad 공통 pagination 적용 * [🛠️ Refactor] 잘못 작성된 읽지 않은 댓글 조회 쿼리 올바르게 정정 (#237) * refactor : CommentsResponse에 자신의 목표인지 확인하는 isMyGoal 필드 추가 (#229) * test : CommentsResponse를 통해 자신의 목표인지 확인할 수 있는지 검증하는 테스트 코드 작성 (#229) * refactor : exist new comment 쿼리를 id와 시간으로 검색하도록 변경 (#236) * refactor : 쿼리가 Goal Id를 사용하도록 변경 (#236) * test : 본인의 Goal만 조회하는지 검증하는 테스트 추가 작성 (#236) * feat: pagination 공통 response 추가 + cheering squad 조회를 위한 queryRepository 추가 * feat(#235): comment count 엔티티 추가 + comment create/delete event 생성 * feat(#235): 목표 타임라인 페이징 조회 API 작업 * fix: dev 환경 로그인 callback 시 redirect uri 변경 * chore: apply lint * refactor: CheererRepository 미사용 메서드 제거 * refactor: TriFunction @FunctionalInterface 적용 * refactor(#235): PR 코멘트 반영 * [🌎 Feature] 타임 라인용 목표 조회 API 개발 (#240) * refactor(#235): pagination 공통화 + cheering squad 공통 pagination 적용 * feat: pagination 공통 response 추가 + cheering squad 조회를 위한 queryRepository 추가 * feat(#235): comment count 엔티티 추가 + comment create/delete event 생성 * feat(#235): 목표 타임라인 페이징 조회 API 작업 * fix: dev 환경 로그인 callback 시 redirect uri 변경 * chore: apply lint * refactor: CheererRepository 미사용 메서드 제거 * refactor: TriFunction @FunctionalInterface 적용 * refactor(#235): PR 코멘트 반영 * hotfix(#235): 타인 타임라인 조회 API 추가 * chore: apply klint * [🛠️ Refactor] 타임 라인 API 작업 리팩토링 (#241) * delete: 유저 정보를 활용한 타임라인 조회 API 삭제 * refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일 * refactor: 메서드 로직에 따라 get, find 명칭 구분 * refactor: lamda body 내 생략 가능한 parameter 생략 * refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCountMap) * refactor: associateWith 메서드 활용 * refactor: PaginationResult total값 Long으로 다시 변경 * hotfix: dev login redirect url 수정 * hotfix: login redirect url referer 기준 분기 처리 * hotfix: login redirect url referer 기준 분기 처리 조건 추가 * hotfix: login redirect url referer 기준 분기 처리 제거 * [🌎 Feature] 타임 라인용 목표 조회 API 개발 - Cursor Deadline으로 변경 (#242) * delete: 유저 정보를 활용한 타임라인 조회 API 삭제 * refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일 * refactor: 메서드 로직에 따라 get, find 명칭 구분 * refactor: lamda body 내 생략 가능한 parameter 생략 * refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCountMap) * refactor: associateWith 메서드 활용 * refactor: PaginationResult total값 Long으로 다시 변경 * fix: 목표 피드 조회 API deadline 역순으로 정렬 조회 * chore: apply lint * fix: goals 테이블의 index 수정 * refactor: pagination cusorId type Any -> Generic으로 변경 * [🌎 Feature] 타임 라인용 목표 조회 API 개발 - Pagination 방식 offset 기반으로 변경 (#243) * delete: 유저 정보를 활용한 타임라인 조회 API 삭제 * refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일 * refactor: 메서드 로직에 따라 get, find 명칭 구분 * refactor: lamda body 내 생략 가능한 parameter 생략 * refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCountMap) * refactor: associateWith 메서드 활용 * refactor: PaginationResult total값 Long으로 다시 변경 * fix: 목표 피드 조회 API deadline 역순으로 정렬 조회 * chore: apply lint * fix: goals 테이블의 index 수정 * refactor: pagination cusorId type Any -> Generic으로 변경 * fix: 타임 라인 목표 조회 API paging 처리 방식 offset으로 변경 * refactor: cursor 기반 pagination 공통 모듈 cursor type long으로 통일 * refactor: pagination 공통 모듈 cursor와 offset 구분 * refactor: 미사용 코드 및 의미 없는 중복 코드 삭제 * hotfix: global exception config 내 NoSuchElementException 설정 수정 * [🚌 Issue] dev api 로그인 요청 위치에 따라 redirect url 변환 (#245) * delete: 유저 정보를 활용한 타임라인 조회 API 삭제 * refactor: PaginationResult.transform() 사용 방식 lamda 형태로 통일 * refactor: 메서드 로직에 따라 get, find 명칭 구분 * refactor: lamda body 내 생략 가능한 parameter 생략 * refactor: 메서드명 내 부정확한 단어 제거(findGoalTimelineCountMap -> findGoalCountMap) * refactor: associateWith 메서드 활용 * refactor: PaginationResult total값 Long으로 다시 변경 * fix: 목표 피드 조회 API deadline 역순으로 정렬 조회 * chore: apply lint * fix: goals 테이블의 index 수정 * refactor: pagination cusorId type Any -> Generic으로 변경 * fix: 타임 라인 목표 조회 API paging 처리 방식 offset으로 변경 * refactor: cursor 기반 pagination 공통 모듈 cursor type long으로 통일 * refactor: pagination 공통 모듈 cursor와 offset 구분 * feat(#244): dev api 로그인 요청 위치에 따라 redirect url 변환 * chore: apply lint * fix: @value(spring.profiles.active) default값 추가 * refactor: 코드 리뷰 반영(#245) * chore: encrypt하던 데이터 삭제 * chore: encrypted test properties 삭제 * chore: referer 저장/조회 logging 추가 * chore: logging 제거 * fix: getReferer 메서드 referer값 null일 경우 Empty String 반환 * [🆘 Bug] 타임라인 관련 버그 수정 (#248) * fix: timeline 조회 시 목표 없으면 early return * feat: 유저정보 기반 타임 라인 조회 API 작업 * fix: 목표 삭제 시 LifeMapCount goalCount 필드 -1 처리 * refactor: timeline 조회 api 내 목표 개수 조회 쿼리 변경 * feat(#171): edit comment count * chore: ktlint * refactor : Comment와 Emoji에 Goal Delete Casacade 속성 추가 (#246) (#249) * refactor : 자신의 Goal에 달린 Comment 삭제를 허용하도록 변경 (#251) (#252) * bug(#254): 타임라인 조회 API 내 코멘트 개수 조회 쿼리 오류 수정 * feat : 이모지 추가 전에 같은 이모지가 존재하는지 확인하는 메서드 작성 (#253) (#256) * refactor : 모든 Emoji 조회 API에서 일관되게 Emoji id 순서로 정렬해 응답하도록 변경 (#257) * refactor : 불필요한 정렬 제거 - MySQL Clustered Index로 인해 이미 pk 정렬 되어 있다 (#257) * fix : 피드 조회 (#259) * feat(#171): 목표 생성시 기본 댓글 개수 추가 * chore(#171): ktlint * feat(#171): 기본 카운트 추가 * chore: ktlint * refact(#171): 변수 삭제 --------- Co-authored-by: ManHyuk <[email protected]> Co-authored-by: binary_ho <[email protected]>
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.
📒 Issue
#171
🎯 어떤 작업을 했는지
피드 조회시 댓글 수가 없는 경우 조회가 안되는데
목표 생성시 기본 댓글 수 넣어주도록 수정
📜 자세한 설명
🌍 영향범위 (모듈)
Resolves: # See also: #