[FEATURE #22]: comment 도메인 기본 구현 - #23
Conversation
- 작성 - 삭제 - 수정 - 단건 조회 - 리스트 조회 (페이지네이션)
- 댓글 작성 - 댓글 수정 - 댓글 삭제 - 내 댓글 조회(무한 스크롤) - 게시글 댓글 조회
Summary of ChangesHello @seoyeon2001, 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! 이 풀 리퀘스트는 댓글 도메인의 핵심 기능을 구현하여 사용자가 게시글에 댓글을 작성, 수정, 삭제하고 자신의 댓글 및 특정 게시글의 댓글을 조회할 수 있도록 합니다. 또한, 관련 게시글 도메인의 기본적인 CRUD 기능과 데이터 유효성 검사, 표준화된 API 응답 및 오류 처리 메커니즘을 도입하여 애플리케이션의 기반 구조를 강화합니다. 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
|
There was a problem hiding this comment.
Code Review
이 PR은 댓글 도메인의 기본 CRUD 기능을 구현하고 관련 게시글 기능도 추가했습니다. 전반적으로 코드가 잘 작성되었고, 특히 댓글 조회 시 삭제된 댓글 처리나 계층 구조를 고려한 정렬 로직이 인상적입니다. 몇 가지 개선점을 제안합니다.
getMyComments의 무한 스크롤 구현에서hasNext로직을 더 견고하게 개선할 수 있습니다.getPostCommentsAPI에 페이지네이션을 추가하여 성능 문제를 예방해야 합니다.- 엔티티에서 타임스탬프를 설정하는 방식이 일관되지 않은 부분이 있어 통일이 필요합니다.
- 게시글 삭제 시 관련 댓글을 처리하는 로직을 구현하여 데이터 정합성을 높이는 것이 좋습니다.
자세한 내용은 각 파일의 리뷰 코멘트를 참고해주세요.
Answl
left a comment
There was a problem hiding this comment.
수고하셨습니다!
컨트롤러는 추후에 인증 끝낸 후에 수정하면 될 것 같습니다
- 부모 댓글로 1차 페이징 - 부모 댓글 기반 2차 페이징
|
/gemini review |
khyun9807
left a comment
There was a problem hiding this comment.
고생하셨습니다!!! 그 컨트롤러 레이어 하드코딩되어있는 사용자 id 1L만
인자로 들어온 AuthenticationPricipal을 이용하는걸로 바꿔 주시면 바로 머지해도 될거 같아요.
머지 되면 전체 swagger작업 들어가겠습니다!
📌 개요
Comment 도메인에 대한 CRUD 기능 API를 구현했습니다.
🔧 작업 내용
✅ 체크리스트
📝 기타 참고 사항
📎 관련 이슈
Close #22