Skip to content
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

Merged
merged 8 commits into from
Feb 4, 2024
Merged

feat: 댓글/답글 CRD API 구현 #54

merged 8 commits into from
Feb 4, 2024

Conversation

kdkdhoho
Copy link
Collaborator

@kdkdhoho kdkdhoho commented Feb 3, 2024

Description

  • 댓글 CRD API 구현했습니다.
    테스트까지 하려면 오늘 내로 못할 거 같고 체력과 정신력이 못버틸 거 같아 일단 PR 올립니다 ,,

  • 리뷰하실 때 중점적으로 봐주실 것은 댓글 조회 로직입니다.
    QueryDsl이 아직 익숙치 않아 CustomCommentRepositoryImpl에서 댓글 조회하는 메서드를 잘 짰는지 모르겠네요.
    그리고 Map<Comment, List<Reply>>로 변환하는 과정에서 댓글의 순서가 보장되지 않을 것 같다는 생각이 들긴 했는데, 테스트를 하지 않아 확신이 서지 않네요.
    만약 정수님이 보셨을 때도 순서 보장이 안돼서 로직 수정이 필요하다 싶으면 리뷰 남겨주세요.

  • 전 이제 지쳤어요 땡벌ㅜ

Relation Issues

@kdkdhoho kdkdhoho added the 기능 label Feb 3, 2024
@kdkdhoho kdkdhoho self-assigned this Feb 3, 2024
@kdkdhoho kdkdhoho requested a review from pparkjs as a code owner February 3, 2024 10:26
@kdkdhoho kdkdhoho linked an issue Feb 3, 2024 that may be closed by this pull request
5 tasks
@kdkdhoho kdkdhoho changed the title feat: 댓글 CRD API 구현 feat: 댓글/답글 CRD API 구현 Feb 3, 2024
@Nahyun-Kang
Copy link

Nahyun-Kang commented Feb 3, 2024

기다리다 지쳤어요 땡벌땡벌~

장난이고 동호님 API 조금 여유 가지고 주셔도 되니 너무 부담가지지 않으셔도 됩니다 ㅠ
전 이제 지쳤어요 땡벌ㅜ 보고 놀라서 댓 남겨봅니다..🥹 남은 주말 시간엔 꼭 쉬어주세요
(참고로 백엔드 깃헙도 PR 제목 보러 온거지 절대 API 진행여부 보러 들어온 거 아님요❌❌❌❌)

Copy link
Collaborator

@pparkjs pparkjs left a comment

Choose a reason for hiding this comment

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

댓글 CRD 구현하시느라 정말 고생많으셨습니다!
동호님이 요청하신 순서 문제와 제안사항 부분 작성했으니 확인 해주시면 감사하겠습니다 ^^

@kdkdhoho
Copy link
Collaborator Author

kdkdhoho commented Feb 4, 2024

@Nahyun-Kang 나현님.. 감사합니다🥹 사알짝 감동받았어요 ㅜ

Copy link
Collaborator

@pparkjs pparkjs left a comment

Choose a reason for hiding this comment

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

수정사항 잘 확인했습니다 수고하셨어요 ㅎㅎ

@pparkjs pparkjs merged commit baf3523 into dev Feb 4, 2024
1 check passed
@pparkjs pparkjs deleted the feat/43 branch February 4, 2024 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

댓글/답글 CRD API 구현
3 participants