-
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 : 리뷰 작성 #45
The head ref may contain hidden characters: "24-be-\uB9AC\uBDF0-\uC791\uC131"
Feat : 리뷰 작성 #45
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.
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.
application.yml 커밋 안해주신 센스...👍 제 리뷰 확인만 해주시면 될 것 같습니다!! 수고하셨습니다~
|
||
@PostMapping | ||
public BaseResponse<PostReviewResponse> createReview(@RequestBody PostReviewRequest postReviewRequest) { | ||
Long reviewId = reviewService.createReview(postReviewRequest.getFacilityId(), |
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.
헤더에 토큰 받는 부분 누락된 것 같습니다!
|
||
import static jakarta.persistence.FetchType.LAZY; | ||
|
||
@Entity | ||
@Getter | ||
@Setter |
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.
도메인에는 세터를 보통 열어두지 않습니다! 메소드를 새로 만드셔서 작성해주시는게 좋아요! 어제 희진님 디코로 리뷰해드리면서 말씀드린 부분인데 단톡에 update 함수 사진 하나 올려둔거 있을거에요 참고하셔서 수정해주시면 좋을 것 같습니다~
Review review = new Review(); | ||
review.setFacility(facility); | ||
review.setRanking(ranking); | ||
review.setComment(comment); |
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.
위에서 말한 것 처럼 도메인에 세터를 지우면 이 부분이 도메인에 작성된 메소드를 부르고 그 안에 값들을 집어넣는 식으로 변경되어야 할 것 같습니다!
…mberId도 저장되도록 수정
감사합니다 ~! 전부 확인후 수정하였습니다 ! |
테스트시 위 쿼리문을 추가해서 확인해주세요 ! |
요약 (Summary)
(application.yml은 commit하지 않았습니다)
🔑 변경 사항 (Key Changes)
📝 리뷰 요구사항 (To Reviewers)
확인 방법
DB에 우선 예시용 facility 를 하나 만듭니다.
POST방식으로 /review 로 요청을 보냅니다. 헤더에 auth필요합니다.
request body 에는 밑에 정보들이 필요하며 facilityId는 반드시 DB에 있는 값을 사용합니다.
다음과같이 response가 반환되는지 확인합니다
review 테이블에 잘 저장되었는지 확인합니다