-
Notifications
You must be signed in to change notification settings - Fork 0
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] 6주차 과제 제출 #8
base: main
Are you sure you want to change the base?
Conversation
6fb9394
to
ab6b074
Compare
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.
고생하셨습니다 :>
나중에 기회되실 때 MemberService에서 직접 토큰을 발급하는 것을 구현하고 그 토큰을 이용해서 헤더에 담아 API 요청을 보내보시면 좋을 것 같아요!
시험 기간 화이팅!
public void deleteByIdV2(Long postId) { | ||
try { | ||
Post post = postJpaRepository.findById(postId) | ||
.orElseThrow(() -> new BusinessException("해당하는 게시글이 없습니다.")); |
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.
P1
시간 되실 때 캐치 예외로 나중에 변경하시면 좋을 것 같아요! :->
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.
수고하셨습니다 ~~ 👍
✒️ 관련 이슈번호 ✒️
💡 과제 내용 및 배운 점 💡
📢 To Reviewers 📢
기본 과제만 했고,, 많은 생각을 못했습니다,, 반성하겠습니다,,