-
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
[Feat] Comment 작성, 수정, 삭제 구현 #7
Conversation
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.
전체적으로 깔끔하게 잘 짜주신 것 같아요! 코멘트 남긴 부분만 다음 commit에 수정해주시면 좋을 것 같습니다 다른 분들도 approve 남기시면 merge 할게요 고생하셨습니다~😊
import lombok.Data; | ||
|
||
@Data | ||
public class CommentUpdateReq { |
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.
CommentUpdateReq 보다 UpdateCommentReq 쪽으로 수정하면 좋을 것 같아요! CommentWriteReq도요!
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.
확인했습니다 ~
@Transactional | ||
public ResponseEntity<?> writeComment(UserPrincipal userPrincipal, CommentWriteReq commentWriteReq) { | ||
|
||
Optional<Member> optionalMember = memberRepository.findById(userPrincipal.getId()); |
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.
이 부분도 optionalMember -> memberOptional 쪽으로 수정하면 좋을 것 같습니다:)
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.
댓글 작성, 수정, 삭제 API 확인했습니다 수고 많으셨습니다~!
}) | ||
@PostMapping | ||
public ResponseEntity<?> writeComment( | ||
@Parameter(description = "AccessToken을 입력해주세요.", required = true) @CurrentUser UserPrincipal userPrincipal, |
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.
LGTM!
✨ PR 유형
어떤 변경 사항이 있나요??
🛠️ 작업내용
Comment 작성, 수정, 삭제 API 구현
📋 추후 진행 상황
위도, 경도 별 Comment 조회 API 구현 중
Store API 구현 예정
📌 리뷰 포인트
빌더를 제대로 사용해서 코드를 구현한 것이 처음이라 다른 분들 코드 참고해서 했습니다
수정이 필요한 부분이 있다면 말씀해 주세요 감사합니다!
✅ Checklist
PR이 다음 요구 사항을 충족하는지 확인해주세요