Skip to content

Conversation

@ayoung-dev
Copy link
Collaborator

@ayoung-dev ayoung-dev commented Dec 8, 2024

📌 과제 설명

👩‍💻 요구 사항과 구현 내용

  • 커뮤니티 게시글 키워드 검색 조회 (제목, 내용)
  • 봉사활동 모집글 키워드 검색 조회 (제목, 내용)
  • 봉사활동 위치기반 모집글 키워드 검색 조회 (제목, 내용)

✅ PR 포인트 & 궁금한 점

  • 봉사활동의 경우 기관명도 같이 조회하고 싶었는데 테이블이 한 번 조인되어있어 머리가 안돌아가서 추후에 리팩토링 하겠습니다..
  • 테스트는 elastic search에서 검색 결과 조회 해오는 부분이 개별로 실행해야만 성공해서 따로 두었습니다.
  • 코드가 좀 더러운 거 같은데 더 못보겠어서 pr 올립니다 아무거나 다 코드 리뷰로 말해주세요..
  • 어제 코드 리뷰 반영하여 다시 pr 올립니다
  • 지도에서 검색 추가 구현하였습니다.
  • 키워드에 값이 없을 경우 전체 조회되도록 수정하였습니다.
  • elastic search랑 연결된 조회는 전부 따로 test를 실행해야하는 건지.. 전체로 돌리면 안돼서 주석처리해뒀습니다. 전부 개별로 실행하면 성공합니다..!
  • 서버 연결하고 다시 pr 올리겠습니다!

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

빌드에 실패했습니다.

@github-actions github-actions bot changed the title Feature/137 Feature/137 봉사활동 모집글, 커뮤니티 게시글 검색 조회 (elastic search) [BUILD FAIL] Feature/137 Feature/137 봉사활동 모집글, 커뮤니티 게시글 검색 조회 (elastic search) Dec 8, 2024
@github-actions github-actions bot closed this Dec 8, 2024
Copy link
Collaborator

@m-a-king m-a-king left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 쿼리 고수가 되셨네요.

테스트 해결해야 할 텐데 왜 안되는 걸까요?
그리고 테스트에서 뭔가 매직 넘버들이 많은데 따라가는 게 조금 헷갈리는 것 같았어요.

Comment on lines +145 to +156
private List<CommunityBoardDocument> convertEntityToDocuments(List<CommunityBoard> communityBoards) {
List<CommunityBoardDocument> communityBoardDocuments = new ArrayList<>();

for (CommunityBoard communityboard : communityBoards) {
CommunityBoardDocument document = CommunityBoardDocument.builder()
.id(communityboard.getId())
.title(communityboard.getTitle())
.content(communityboard.getContent())
.build();
communityBoardDocuments.add(document);
}
return communityBoardDocuments;
Copy link
Collaborator

Choose a reason for hiding this comment

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

서비스에 있으면 좋을 것 같은데 혹시 여기 있는 이유가 있나요?

Copy link
Collaborator Author

@ayoung-dev ayoung-dev Dec 9, 2024

Choose a reason for hiding this comment

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

documentRepository의 saveAll 사용하려고 repository에서 document로 바꿔주었습니다!

this.title = title;
this.content = content;
}
} No newline at end of file
Copy link
Collaborator

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] 커뮤니티 게시글 / 봉사 활동 모집글 elasticsearch 검색

3 participants