-
Notifications
You must be signed in to change notification settings - Fork 1
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
Merge Develop To Main #261
Merged
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
* 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(#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에서 제거
* chore : 리팩토링 이슈 템플릿 세팅 * chore : 기타-이슈 기본 생성 제목에 불필요한 띄어쓰기 제거
* refactor : IsMyReaction을 확인할 때 Username이 아닌 id로 확인하도록 변경 (#217) * merge with develop
* feat(#209): spring security authorization_request_not_found 수정 * fix: AuthorizationRequestRepository cookie 기반에서 state parameter 기반으로 변경 * fix: AuthorizationRequestRepository remove 메서드에 명시적인 삭제 추가
…SecurityCustomizer 제거 (#226) * refactor: RunnerLock 내부 필드 변수명 변경 * refactor: security permitall uri 상수화
[🌎 Feature] lifemap 응답 필드 수정 #232
* 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: 미사용 코드 및 의미 없는 중복 코드 삭제
* 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 추가
* fix: timeline 조회 시 목표 없으면 early return * feat: 유저정보 기반 타임 라인 조회 API 작업 * fix: 목표 삭제 시 LifeMapCount goalCount 필드 -1 처리 * refactor: timeline 조회 api 내 목표 개수 조회 쿼리 변경
/goal/explore 댓글 수 수정
[🆘 Bug] 타임라인 조회 API 내 코멘트 개수 조회 쿼리 오류 수정
[🛠️ Refactor] 이모지 응답시 고정된 순서로 내려주기
목표 추가시 기본 카운트 추가
ManHyuk
approved these changes
Apr 26, 2024
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.
👍
🚀 🚀 🚀 🚀 |
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.
No description provided.