[feat #45] 게시글 상세 조회 응답 필드 추가 #46
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
3 tasks
Test Results64 tests 58 ✅ 6s ⏱️ Results for commit ee5ca90. ♻️ This comment has been updated with latest results. |
|
작업하다 시간이 늦어졌네요.. 내일 알바 다녀와서 오후 중으로 리뷰 진행하겠습니다 |
dudxo
approved these changes
Aug 16, 2024
src/main/java/com/dnd/gongmuin/question_post/dto/QuestionPostMapper.java
Outdated
Show resolved
Hide resolved
| @@ -56,4 +71,11 @@ public PageResponse<QuestionPostSimpleResponse> searchQuestionPost( | |||
| .map(QuestionPostMapper::toQuestionPostSimpleResponse); | |||
| return PageMapper.toPageResponse(responsePage); | |||
| } | |||
|
|
|||
| private int getCountByType(Long questionPostId, InteractionType type){ | |||
There was a problem hiding this comment.
totalCount 필드를 가져오는거니 getTotalCountByType은 어떠신가요?
There was a problem hiding this comment.
혹시 그러면 interactionCount엔티티에서 필드명을 count로 바꾸는건 어떨까요? 엔티티명도 interactionCount니까 total count보다 count가 더 명확하지 않을까라는 생각이 갑자기 드네요,,,
There was a problem hiding this comment.
음..저는 차리리 엔티티명을 InteractionTotalCount로 바꾸는거 더 맞지않았나 싶습니다.
�count보다 TotalCount가 더 정확한 표현 같거든요
dudxo
pushed a commit
that referenced
this pull request
Aug 16, 2024
* [feat] : 질문글 등록 응답 dto 생성 * [feat] : 상세 조회 응답 dto에 추천수, 북마크수 필드 추가 * [feat] : 상세 조회 DTO, 등록 DTO 분리 * [feat] : 북마크수, 추천수 구하는 비즈니스 로직 추가 * [refactor] : fixture 메서드명 수정 * [feat] : 북마크수, 추천수 구하는 비즈니스 로직 테스트 * [feat] : API별 DTO 분리 반영 * [test] : dto 응답 필드 추가 반영 * [style] : 코드 리포멧팅 * [fix] : DTO mapper 함수명 수정 * [feat] : 엔티티 필드명 수정
dudxo
added a commit
that referenced
this pull request
Aug 16, 2024
* [feat] : MemberQueryDSL Repo 추가 * [feat] : 작성한 질문 전체 조회 응답 DTO 추가 * [feat] : MemberErrorCode 추가 * [feat] : 작성한 질문 전체 조회 컨트롤러/비즈니스 로직 작성 * [test] : 특정 회원 작성 질문 목록 검색 테스트 * [feat] : 댓글 단 질문 목록 응답 DTO 추가 * [feat] : 댓글 단 질문 목록 QueryDSL 추가 * [feat] : 댓글 단 질문 목록 에러코드 추가 * [feat] : 댓글 단 질문 전체 조회 컨트롤러/비즈니스 로직 추가 * [test] : 특정 회원 답변 작성 질문 목록 리스트 조회 테스트 * [test] 마이페이지 - 나의활동 API 통합 테스트 추가 * [feat #38] 질문글 상호작용 API (#44) * [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 #45] 게시글 상세 조회 응답 필드 추가 (#46) * [feat] : 질문글 등록 응답 dto 생성 * [feat] : 상세 조회 응답 dto에 추천수, 북마크수 필드 추가 * [feat] : 상세 조회 DTO, 등록 DTO 분리 * [feat] : 북마크수, 추천수 구하는 비즈니스 로직 추가 * [refactor] : fixture 메서드명 수정 * [feat] : 북마크수, 추천수 구하는 비즈니스 로직 테스트 * [feat] : API별 DTO 분리 반영 * [test] : dto 응답 필드 추가 반영 * [style] : 코드 리포멧팅 * [fix] : DTO mapper 함수명 수정 * [feat] : 엔티티 필드명 수정 * [fix] : PostInteractionCount -> InteractionCount 네이밍 변경으로 인한 오류 수정 * [test] : repoisitory 테스트 실패 수정 * [fix] : 게시글 스크랩/추천해요 총 개수 제대로 매핑되도록 쿼리 수정 * [fix] : 작성 질문/답변 단 질문 응답 DTO 스크랩/추천해요 충 개수 필드 수정 * [test] : 스크랩/추천해요 총 개수에 관한 테스트 추가 * [feat] : 답변 단 질문 목록 조회시 가장 최신 답변 1개만 가져오도록 쿼리 추가 * [test] : 답변 단 질문 목록 조회 관련 Repostiory/Controller 테스트 작성 * [fix] : 연산 타입 일치화(+1 -> +1L) * [fix] : 질문 목록 id 값이 아닌 updatedAt() 내림차순으로 정렬 변경 --------- Co-authored-by: Son Gahyun <[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.
관련 이슈
📑 작업 상세 내용
💫 작업 요약