[feat #41] 나의활동 API #47
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
Closed
2 tasks
Test Results72 tests 67 ✅ 7s ⏱️ Results for commit 64df7d3. ♻️ This comment has been updated with latest results. |
|
@hyun2371 비즈니스 로직 응답 부분에 문제가 있는 것 같아서 검증 및 수정 진행 후 재요청드리겠습니다! ++ CI build 오류 수정 |
* [style] : 정적 팩토리 메서드 네이밍 변경 * [style] : enum 라벨 필드명 변경 * [feat] : 게시글 상호작용 에러코드 추가 * [feat] : postInteraction mapper 추가 * [feat] : postInteraction 상태 변경 메서드 추가 * [feat] : postInteractionCount 필드 변경 메서드 추가 * [feat] : 게시글 상호작용 응답 추가 * [feat] : 게시글 상호작용, 게시글 상호작용수 repository 추가 * [feat] : 게시글 상호작용(추천, 북마크) 활성화, 비활성화 비즈니스 로직 작성 * [feat] : 게시글 상호작용(추천, 북마크) 활성화, 비활성화 API 메서드 작성 * [feat] : 상호작용 에러 코드 추가 및 네이밍 변경 * [refactor] : 질문글 비즈니스 메서드 파라미터 변경 * [feat] : 질문글 상호작용 생성 전 검증 로직 추가 * [test] : 상호작용, 상호작용 수 fixture 추가 * [fix] : 상태변경 로직 오류 수정 * [feat] : 상태변경 에러코드 수정 * [feat] : 응답 필드에 라벨로 반환 * [test] : 상호작용 활성화/비활성화 단위 테스트 * [style] : 코드 리포멧팅 * [feat] : 컨트롤러 파라미터 어노테이션 수정 * [feat] : 게시글 상호작용 생성 시 기존에 게시글 상호작용 수 엔티티 존재하는지 검증 * [test] : 게시글 상호작용 생성 시 기존에 게시글 상호작용 수 엔티티 존재하는지 로직 추가에 대한 테스트 반영 * [test] : 이메일 중복 저장 방지를 위해 memberFixture 객체 추가 * [fix] : 도메인 비즈니스 로직 오류 수정 * [test] : 게시글 상호작용 통합 테스트 작성 * [rename] : 상호작용, 상호작용 수 엔티티명 변경 * [style] : 코드 리포멧팅
* [feat] : 질문글 등록 응답 dto 생성 * [feat] : 상세 조회 응답 dto에 추천수, 북마크수 필드 추가 * [feat] : 상세 조회 DTO, 등록 DTO 분리 * [feat] : 북마크수, 추천수 구하는 비즈니스 로직 추가 * [refactor] : fixture 메서드명 수정 * [feat] : 북마크수, 추천수 구하는 비즈니스 로직 테스트 * [feat] : API별 DTO 분리 반영 * [test] : dto 응답 필드 추가 반영 * [style] : 코드 리포멧팅 * [fix] : DTO mapper 함수명 수정 * [feat] : 엔티티 필드명 수정
hyun2371
reviewed
Aug 16, 2024
| public Slice<QuestionPostsByMemberResponse> getQuestionPostsByMember(Member member, Pageable pageable) { | ||
| QQuestionPost qp = QQuestionPost.questionPost; | ||
| QInteractionCount ic = QInteractionCount.interactionCount; | ||
|
|
There was a problem hiding this comment.
개인적으로 따로 변수로 빼야하나 생각이 듭니다! 변수명를을축약하니까 한 번 더 생각을 해야 하는 것 같아요! static import를 하면 별로 길어질 것 같지 않습니다.
There was a problem hiding this comment.
기존 가현님 코드를 참고하며 static import를 하다보니 무언가 더 이해하기 힘들다라는 생각이 들더라고요!
이 부분은 추후 더 생각해보겠습니다!
There was a problem hiding this comment.
QQuestionPost가 static import 해서 questionPost로 되는거라 이해가 안된다는 말씀이 잘 와닿지 않네요,, 저는 오히려 개인적으로 약어를 변수명으로 선언한게 한 번 더 생각하도록 만든다고 느꼈습니다!
src/main/java/com/dnd/gongmuin/member/repository/MemberCustomImpl.java
Outdated
Show resolved
Hide resolved
src/main/java/com/dnd/gongmuin/member/dto/response/QuestionPostsByMemberResponse.java
Outdated
Show resolved
Hide resolved
src/main/java/com/dnd/gongmuin/member/dto/response/AnsweredQuestionPostsByMemberResponse.java
Outdated
Show resolved
Hide resolved
src/main/java/com/dnd/gongmuin/member/dto/response/QuestionPostsByMemberResponse.java
Show resolved
Hide resolved
|
@hyun2371 getAnsweredQuestionPostsByMember() 쿼리가 매우 길고 더티한데 시간이 없어 리팩토링 때 변경하겠습니다ㅜㅜ |
hyun2371
approved these changes
Aug 16, 2024
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.
관련 이슈
📑 작업 상세 내용
NULL일 수 있어서 Inner Join이 아닌 Left Join 사용NULL일 수 있어서 Inner Join이 아닌 Left Join 사용💫 작업 요약
🔍 중점적으로 리뷰 할 부분