[4주차] 김지훈 /[feat] 추가 API 구현#160
Open
jihoonkim501 wants to merge 17 commits intoLeets-Official:김지훈/mainfrom
Hidden character warning
The head ref may contain hidden characters: "\uae40\uc9c0\ud6c8/4\uc8fc\ucc28"
Open
[4주차] 김지훈 /[feat] 추가 API 구현#160jihoonkim501 wants to merge 17 commits intoLeets-Official:김지훈/mainfrom
jihoonkim501 wants to merge 17 commits intoLeets-Official:김지훈/mainfrom
Conversation
gusanans218
approved these changes
Apr 30, 2026
gusanans218
left a comment
gusanans218
left a comment
There was a problem hiding this comment.
댓글 관련 로직을 controller / service / repository / entity / dto 단위로 잘 분리하신 점이 좋았습니다!! Post 도메인과 유사한 구조로 확장되어서 전체 코드베이스의 일관성이 잘 유지되는 것 같아요 ㅎㅎ
| POST_ALREADY_DELETED(HttpStatus.BAD_REQUEST,"POST_400","이미 삭제된 게시글입니다."); | ||
| POST_NOT_FOUND(HttpStatus.NOT_FOUND,"POST_404","게시글이 존재하지 않습니다."), | ||
| POST_ALREADY_DELETED(HttpStatus.BAD_REQUEST,"POST_400","이미 삭제된 게시글입니다."), | ||
| FORBIDDEN_COMMENT_UPDATE(HttpStatus.FORBIDDEN,"POST_403","게시글을 수정할 권한이 없습니다."), |
There was a problem hiding this comment.
FORBIDDEN_COMMENT_UPDATE는 댓글 관련 에러라서 Post 에러들과 함께 있는 것보다 Comment 영역으로 분리하면 더 읽기 좋을 것 같아요! 현재 code/message도 게시글 기준으로 되어 있어 댓글 수정 권한 문구로 맞추면 좋을 것 같습니다!!
N-yujeong
approved these changes
Apr 30, 2026
N-yujeong
left a comment
N-yujeong
left a comment
There was a problem hiding this comment.
👍 좋아요/신고 기능의 중복 방지 로직을 ErrorStatus에 잘 반영해주셨습니다.
다만, FORBIDDEN_COMMENT_UPDATE는 Comment 영역의 에러인데 Post 영역에 분류되어 있네요. Comment 에러 섹션으로 이동하면 가독성이 더 좋을 것 같습니다!
Hanharam
approved these changes
Apr 30, 2026
Hanharam
left a comment
Hanharam
left a comment
There was a problem hiding this comment.
👍 전체적으로 깔끔한 코드와 validator를 따로 분리한 점이 좋았습니다!
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
1. 과제 요구사항 중 구현한 내용
2. 핵심 변경 사항
기본 API는 각설하고 게시글 좋아요 , 좋아요 취소 기능 ( 게시글, 댓글)을 구현하였습니다. 요구사항에 따라 좋아요 상태에서 다시 좋아요를 누르는, 좋아요가 되어있지않은 상태에서 좋아요 취소하는 상황을 구현체에서 예외처리를 하였고 신고 기능 또한 추가하였습니다. 일정 신고 갯수가 누적되면 status를 변화하여 접수 , 삭제 or 숨김 상태로 전환되도록 구현하였고 soft delete 형식을 사용하기위해 추후에 scheduler를 도입하여 관리할 생각입니다.
3. 실행 및 검증 결과
!! 기본적인 CRUD 캡쳐는 제외하고 올립니다.
게시글 Stauts가 HIDDEN 상태로 전환.
댓글 또한 이와 같습니다.
4. 완료 사항
게시글, 댓글 좋아요,신고 상태 변환에 대한 예외처리 및 기능 구현 성공
5. 추가 사항
제출 체크리스트
{이름}/main브랜치다{이름}/{숫자}주차브랜치다