Feat: 신고하기 API 구현 #153
Open
Feat: 신고하기 API 구현 #153
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
seoyeonjin
reviewed
Apr 5, 2025
Collaborator
seoyeonjin
left a comment
seoyeonjin
left a comment
There was a problem hiding this comment.
신고 타입을 Enum으로 만드는 과정에서 커뮤니티 신고라면 굳이 ERD 설계처럼 Enum으로 저장이 필요없을 것 같다고 생각했고, Enum을 Report에 저장하게 된다면 이후 리뷰, 댓글, 대댓글 신고 기능이 추가될 때 파라미터만 바꿔서 요청 받으면 하나의 API에서 처리할 수 있다고 생각했습니다! 이 부분에 대한 연진님의 의견은 어떠신지가 궁금합니다!
이 부분 api에서도 enum으로 처리하도록 변경한다는 얘기가 맞을까요? api endpoint 변경한 점 좋습니다! 확장도 더 용이할 것 같습니다.
| private final ReportRepository reportRepository; | ||
|
|
||
| @Transactional | ||
| public void addReport(final Long memberId, final Long targetId, final ReportType reportType) { |
Collaborator
There was a problem hiding this comment.
p2;
지금은 해당하는 post id가 존재하지 않아도 해당 post에 대해 report가 가능한 구조인데, 해당 id가 유효한 id인지 확인하는 로직이 추가되면 좋을 것 같습니다!
Member
Author
There was a problem hiding this comment.
이 부분에 대해서 Controller 단에서 커스텀 어노테이션으로 구분을 할지, 아니면 서비스 단에서 처리를 할지 등등도 논의하면 좋을 것 같습니다!
Member
Author
네네! Converter를 추가를 하지 않은 것 같은데 추가하도록 하겠습니다! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔥Pull requests
⛳️ 작업한 브랜치
👷 작업한 내용
🚨 참고 사항
신고 타입을 Enum으로 만드는 과정에서 커뮤니티 신고라면 굳이 ERD 설계처럼 Enum으로 저장이 필요없을 것 같다고 생각했고, Enum을 Report에 저장하게 된다면 이후 리뷰, 댓글, 대댓글 신고 기능이 추가될 때 파라미터만 바꿔서 요청 받으면 하나의 API에서 처리할 수 있다고 생각했습니다! 이 부분에 대한 연진님의 의견은 어떠신지가 궁금합니다!
위의 고려사항에 따라 다음과 같이 변경을 했습니다.