Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
네이버 컨벤션에 부합하게 개행 처리
SignINVendorRequest에서 검증 수행 후 Service 에 위임
`+` matches()
`+` findByEmail() `+` findByEmailOrThrow()
입력된 비밀번호와 Vendor의 비밀번호의 일치 여부 확인
SignInVendorService는 PasswordMismatchException을 발생시킨다.
부모 클래스에 필드값을 넣어주어야 제대로 동작한다.
# Conflicts: # src/main/java/camp/woowak/lab/vendor/repository/VendorRepository.java # src/main/java/camp/woowak/lab/web/api/vendor/VendorApiController.java
Closed
|
현재 vendor와 관련된 코드만 변경되어있는데, 해당 이슈에서 Customer와 Vendor의 Long타입 ID를 사용하고 있는 다른 Service, DTO, Controller등의 수정을 같이 하시는건가요? 고치지않으면 빌드가 깨질것같아 질문드립니다! |
`+` PasswordMismatchException `+` NotFoundVendorException
Vendor의 id 타입의 변경으로 인한 수정
응답 상태 코드가 BadRequest인지 검증
status 값을 담아주는 APIResponseAdvice 활용
네 현재의 main 브랜치를 pull 받아서 수정 및 빌드 테스트 통과한 상태입니다. 그 과정에서 Store를 생성하는 코드를 수정했습니다. #51 에도 Vendor관련 로직이 있어서 머지된 이후 수정하게 될 것 같아요. 현재 pr은 #51이 처리되기 전까지 잠시 드리프트 하겠습니다 |
kimhyun5u
reviewed
Aug 15, 2024
src/main/java/camp/woowak/lab/vendor/service/SignUpVendorService.java
Outdated
Show resolved
Hide resolved
Service 내부에서 DataIntegrityViolationException을 받기 위해서는 flush가 수행되어야 한다.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
💡 다음 이슈를 해결했어요.
Issue Link - #9
Issue Link - #41
LoginCustomer,LoginVendor또한 UUID를 저장하게 됩니다.💡 이슈를 처리하면서 추가된 코드가 있어요.
LoginMember+LoginCustomer,LoginVendor는 UUID를 저장합니다.💡 다음 자료를 참고하면 좋아요.
The Problem with Using a UUID Primary Key in MySQL
JPA Hibernate Id 생성 전략과 UUID 전략
JPA 에서 UUID 삽질기록
How To Use UUIDs With Hibernate And MySQL
The BINARY and VARBINARY Types
✅ 셀프 체크리스트