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

[OING-375] feat: 음성 댓글 관련 API 목을 추가해요 #275

Merged
merged 3 commits into from
Dec 5, 2024

Conversation

CChuYong
Copy link
Member

@CChuYong CChuYong commented Nov 30, 2024

❓ 기능 추가 배경


음성 댓글 관련 API 목을 추가했어요.

➕ 추가/변경된 기능


  • 음성 댓글 추가, 삭제, 조회 API 목 추가

🥺 리뷰어에게 하고싶은 말


오랜만에 쓰네요 ㅎㅎㅎ

🔗 참조 or 관련된 이슈


https://no5ing.atlassian.net/browse/OING-375

@CChuYong CChuYong added the ✨ FEATURE 기능 추가 label Nov 30, 2024
@CChuYong CChuYong self-assigned this Nov 30, 2024
Copy link

github-actions bot commented Nov 30, 2024

Test Results

 52 files   52 suites   12s ⏱️
144 tests 144 ✅ 0 💤 0 ❌
145 runs  145 ✅ 0 💤 0 ❌

Results for commit 36d88fe.

♻️ This comment has been updated with latest results.

}

@Override
public PaginationResponse<PostVoiceCommentResponse> getPostComments(String postId, Integer page, Integer size,
Copy link
Collaborator

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이라는 플래그 값을 둬서 구분하는 방법을 논의 했었는데, 이 방법을 안쓰고 분리해서 구현하는 방법으로 진행한 이유가 무엇일까요?

Copy link
Member

Choose a reason for hiding this comment

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

@Kwon770 그때 당시에는 제가 요구사항을 깊게 고려하지 않았던 상태라 타입을 둬서 구분하자는 방식으로 논의했었는데 일반 댓글과 음성 댓글을 분리해서 구현하는 게 맞다고 생각합니다. 일반 댓글은 content를 가지고 음성 댓글은 content 없이 presignedurl(audioUrl)을 가지는데 Comment 엔티티 하나로 처리하는 것보다 음성 댓글용 엔티티 하나를 더 두어서 처리하는 게 책임과 역할이 더 명확하다고 생각해요. 혹시 이런 의도로 모킹한게 맞을까요? @CChuYong

Copy link
Member

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를 정렬해서 화면에 보여주는 방식으로 구현이 가능한가요?

Copy link

sonarqubecloud bot commented Dec 5, 2024

Copy link

github-actions bot commented Dec 5, 2024

Code Coverage

File Coverage [3.13%]
VoiceCommentController.java 8.33%
PostCommentResponseV2.java 0%
CommentType.java 0%
Total Project Coverage 52.39% 🍏

Copy link
Member

@Ji-soo708 Ji-soo708 left a comment

Choose a reason for hiding this comment

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

LGTM! 수고하셨습니다~

@Ji-soo708 Ji-soo708 merged commit f3f441b into dev Dec 5, 2024
5 checks passed
@Ji-soo708 Ji-soo708 deleted the feature/OING-375 branch December 5, 2024 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ FEATURE 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants