-
Notifications
You must be signed in to change notification settings - Fork 1
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] Customer id 변경 #58
The head ref may contain hidden characters: "feature/40_kimhyun5u_customer-id-\uBCC0\uACBD"
Conversation
- then().save() -> then().saveAndFlush()
- Customer id: Long -> UUID 변경
- Customer id: Long -> UUID 변경
- getId() 의 반환값 UUID 로 변경
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.
기능상 중요하지는 않은 커맨트라서 승인드립니다.
src/main/java/camp/woowak/lab/customer/service/SignUpCustomerService.java
Outdated
Show resolved
Hide resolved
src/main/java/camp/woowak/lab/web/authentication/LoginMember.java
Outdated
Show resolved
Hide resolved
src/main/java/camp/woowak/lab/customer/service/SignUpCustomerService.java
Outdated
Show resolved
Hide resolved
- getId() 의 반환값 Object 로 변경
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 :)
💡 다음 이슈를 해결했어요.
Issue Link - #40
Discussion Link - #36
💡 이슈를 처리하면서 추가된 코드가 있어요.
💡 이런 고민을 했어요.
✅ 셀프 체크리스트