Skip to content

[BE-198] 검색으로 레코드를 조회하는 API 개발#152

Merged
kdomo merged 9 commits intodevelopfrom
feature/BE-198
Feb 10, 2023
Merged

[BE-198] 검색으로 레코드를 조회하는 API 개발#152
kdomo merged 9 commits intodevelopfrom
feature/BE-198

Conversation

@Pull-Stack
Copy link
Copy Markdown
Contributor

@Pull-Stack Pull-Stack commented Feb 8, 2023

관련 이슈 번호

설명

  • 검색으로 레코드를 조회하는 API를 개발하였습니다.
  • 달력으로 레코드를 조회하는 API는 기존에 개발된 API(마이레코드 조회)를 재사용할 수 있으므로 이번 스프린트에서 따로 개발하지 않았습니다.

변경사항

  • 테스트코드 작성
  • RecordBySearchRequestDto의 searchKeyword필드 NotNull 유효성 검사 추가
  • 정상적인 경우의 테스트코드 추가

질문사항

@Pull-Stack Pull-Stack added ✨ Feature 기능 개발 🔨 Refactor 리팩토링 labels Feb 8, 2023
Copy link
Copy Markdown
Member

@kdomo kdomo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

findRecord를 찾은 후 for문을 돌면서 commentRepository.countByRecordAndParentCommentIsNull(findRecord); 를 사용하여
댓글 개수를 count하는 쿼리가 레코드 개수만큼 발생합니다.
추 후 레코드와 댓글 양방향 매핑으로 수정 고려사항이라 이부분 제외 후 잘 작성해주셨어요 고생하셨습니다

@kdomo kdomo merged commit 21effa3 into develop Feb 10, 2023
@kdomo kdomo deleted the feature/BE-198 branch February 10, 2023 06:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 기능 개발 🔨 Refactor 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants