-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: 댓글/답글 CRD API 구현 #54
Conversation
기다리다 지쳤어요 땡벌땡벌~ 장난이고 동호님 API 조금 여유 가지고 주셔도 되니 너무 부담가지지 않으셔도 됩니다 ㅠ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
댓글 CRD 구현하시느라 정말 고생많으셨습니다!
동호님이 요청하신 순서 문제와 제안사항 부분 작성했으니 확인 해주시면 감사하겠습니다 ^^
src/main/java/com/listywave/list/application/service/CommentService.java
Show resolved
Hide resolved
src/main/java/com/listywave/list/application/service/CommentService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/listywave/list/application/service/CommentService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/listywave/list/application/service/CommentService.java
Show resolved
Hide resolved
src/main/java/com/listywave/list/repository/CustomCommentRepositoryImpl.java
Outdated
Show resolved
Hide resolved
@Nahyun-Kang 나현님.. 감사합니다🥹 사알짝 감동받았어요 ㅜ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정사항 잘 확인했습니다 수고하셨어요 ㅎㅎ
Description
댓글 CRD API 구현했습니다.
테스트까지 하려면 오늘 내로 못할 거 같고 체력과 정신력이 못버틸 거 같아 일단 PR 올립니다 ,,
리뷰하실 때 중점적으로 봐주실 것은 댓글 조회 로직입니다.
QueryDsl이 아직 익숙치 않아
CustomCommentRepositoryImpl
에서 댓글 조회하는 메서드를 잘 짰는지 모르겠네요.그리고
Map<Comment, List<Reply>>
로 변환하는 과정에서 댓글의 순서가 보장되지 않을 것 같다는 생각이 들긴 했는데, 테스트를 하지 않아 확신이 서지 않네요.만약 정수님이 보셨을 때도 순서 보장이 안돼서 로직 수정이 필요하다 싶으면 리뷰 남겨주세요.
전 이제 지쳤어요 땡벌ㅜ
Relation Issues