Conversation
Summary of ChangesHello @jiminnimij, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 전시회 상세 정보 및 게시글(리뷰, 미리보기) 조회 로직을 개선하고, 관련 DTO에 추가 정보를 포함하도록 업데이트합니다. 특히, 삭제된 게시글 중 답변이 있는 경우에도 조회될 수 있도록 쿼리 로직을 변경하여 데이터 표시 유연성을 높였습니다. 또한, ReviewController에서 PreviewService를 도입하여 게시글 조회 로직을 일반화하고 재사용성을 향상시켰습니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이번 PR은 전시 상세 정보에 카테고리를 추가하고, 삭제된 게시글 처리 로직을 개선하며, 게시글 조회 로직을 리팩터링하는 변경 사항을 포함하고 있습니다. 전반적으로 코드 재사용성을 높이려는 시도는 좋았으나, 리팩터링 과정에서 리뷰 목록 조회 API의 응답 형식이 의도치 않게 변경되는 심각한 버그가 발생했습니다. 또한, ExhibitionReview DTO에 새로운 필드를 추가하면서 초기화 로직이 누락된 부분이 있습니다. 자세한 내용은 각 파일의 리뷰 코멘트를 참고해 주세요.
| int limit | ||
| ) { | ||
| return reviewService.getExhibitionReviews(memberId, exhibitionId, pageNum, limit); | ||
| return previewService.getExhibitionPreviews(memberId, exhibitionId, PostType.REVIEW, pageNum, limit); |
There was a problem hiding this comment.
이 변경으로 인해 리뷰 목록 조회 API가 잘못된 데이터 형식(ExhibitionPreview)을 반환하게 됩니다. previewService.getExhibitionPreviews 메서드는 ExhibitionPreview DTO를 생성하도록 구현되어 있어, 리뷰에 포함되어야 할 이미지 정보(images, imageCount)가 누락되고, 불필요한 답변 관련 필드(answer, hasAnswer 등)가 포함됩니다.
리뷰 목록은 ExhibitionReview DTO를 사용해야 합니다. PreviewService 내에 postType에 따라 적절한 DTO를 생성하도록 분기 로직을 추가하거나, 리뷰 목록 조회 로직을 ReviewService에 그대로 두는 것을 고려해 보세요. 이 변경은 API 스펙을 깨뜨리는 심각한 버그입니다.
| private List<String> images; | ||
| private int imageCount; | ||
| @JsonProperty("isDeleted") | ||
| private boolean isDeleted; |
There was a problem hiding this comment.
isDeleted 필드가 추가되었지만, of 정적 팩토리 메서드에서 post 객체의 isDeleted 상태를 반영하여 초기화하는 로직이 누락되었습니다. 이로 인해 API 응답에서 항상 isDeleted가 false로 반환될 수 있습니다. ExhibitionPreview.of() 메서드 구현을 참고하여 isDeleted(post.isDeleted())를 빌더에 추가해 주세요.
1 participant
#️⃣ 연관된 이슈
#️⃣ 작업 내용
#️⃣ 테스트 결과
#️⃣ 변경 사항 체크리스트
#️⃣ 스크린샷 (선택)
#️⃣ 리뷰 요구사항 (선택)
📎 참고 자료 (선택)