-
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
[Fix/#109] 모델 메인뷰에서 상세보기 할시에, isClicked true 처리 #111
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.
offer 로 application 찾기 확인했습니다!
@@ -183,10 +181,22 @@ private StyleDetailResponse getStyleDetailResponse(Long userId, Long offerId) { | |||
return styleDetailResponse; | |||
} | |||
|
|||
@Transactional | |||
public void handleOfferClickStatus(HairServiceOffer hairServiceOffer){ |
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.
p2
해당 클래스 내부에서만 사용하는 함수인 것 같아 private
이 적합해 보입니다!
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.
작업하시느라 고생하셨습니다!
코맨트 확인해주세요!
@@ -183,10 +181,22 @@ private StyleDetailResponse getStyleDetailResponse(Long userId, Long offerId) { | |||
return styleDetailResponse; | |||
} | |||
|
|||
@Transactional |
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.
p3
해당 @Transactional
이 없어도 될 것이라고 생각합니다!
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.
굿굿!!
관련 이슈번호
해결하는 데 얼마나 걸렸나요? (예상 작업 시간 / 실제 작업 시간)
해결하려는 문제가 무엇인가요? && 어떻게 해결했는가요