-
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
[OING-375] feat: 음성 댓글 관련 API 목을 추가해요 #275
Conversation
Test Results 52 files 52 suites 12s ⏱️ Results for commit 36d88fe. ♻️ This comment has been updated with latest results. |
} | ||
|
||
@Override | ||
public PaginationResponse<PostVoiceCommentResponse> getPostComments(String postId, Integer page, Integer size, |
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.
음성 댓글 만을 위한 Mock data가 필요해서 별도의 댓글 조회 api가 필요하다고는 생각하는데, response dto가 음성 댓글 용인 것으로 봐서 일반 댓글과 음성 댓글을 애초에 분리해서 조회하는 API 로 설계된걸로 보여요.
@Ji-soo708 와 음성 댓글 구현에 관해서 짧게 이야기 나눴을 때는 댓글에 TYPE이라는 플래그 값을 둬서 구분하는 방법을 논의 했었는데, 이 방법을 안쓰고 분리해서 구현하는 방법으로 진행한 이유가 무엇일까요?
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.
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.
@CChuYong 다만 지금 구현 방식에서는 comment 조회와 voice-comment 조회가 각각 이루어질텐데 이 둘은 프론트 단에서 두 조회용 API 응답을 합쳐서 id를 정렬해서 화면에 보여주는 방식으로 구현이 가능한가요?
Quality Gate passedIssues Measures |
Code Coverage
|
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.
LGTM! 수고하셨습니다~
❓ 기능 추가 배경
음성 댓글 관련 API 목을 추가했어요.
➕ 추가/변경된 기능
🥺 리뷰어에게 하고싶은 말
오랜만에 쓰네요 ㅎㅎㅎ
🔗 참조 or 관련된 이슈
https://no5ing.atlassian.net/browse/OING-375