-
Notifications
You must be signed in to change notification settings - Fork 8
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
[#199, #198] 환불 API 작성, Payment 도메인 수정 #200
Merged
Merged
Changes from 1 commit
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
c55af3b
refactor: PaymentSaveResponse -> PaymentPendingResponse 네이밍 변경, Payme…
Joyykim 78efff0
feat: Payment 환불 도메인 로직 작성
Joyykim 5bc76ae
feat: Payment 환불 컨트롤러,서비스 로직 작성
Joyykim dd89c27
feat: 환불API js코드예시 작성
Joyykim 4f191a1
Merge branch 'develop-server' of https://github.com/woowacourse-teams…
Joyykim d0a756b
refactor: PaymentService#cancelPayment의 불필요한 로직 정리
Joyykim 1f48d36
refactor: PaymentCancelRequest의 환불금액 필드 제거
Joyykim 6e3d048
test: Payment 도메인 테스트 작성
Joyykim 8b397ce
feat: Payment의 id 타입을 UUID로 변경
Joyykim 6611c32
feat: Payment 관련 DTO의 merchantUid 타입을 UUID로 변경, 더이상 사용되지 않는 DonationR…
Joyykim e79a835
Merge branch 'develop-server' of https://github.com/woowacourse-teams…
Joyykim fe9121f
fix: request dto에서 UUID를 받지 못하는 이슈 해결
Joyykim 8c57263
fix: PaymentNotFoundException 메시지/에러코드 수정, 사용되지 않는 PaymentRequestExce…
Joyykim 6c4c108
refactor: IllegalPaymentInfoException 예외메시지 수정
Joyykim a3f1bcc
fix: 환불요청dto 유효성검사 실패시 PaymentCancelRequestException 발생하도록 변경
Joyykim b83ca46
test: Payment 컨트롤러 테스트케이스 추가
Joyykim 049e8e1
refactor: convertToPaymentInfo 수정
Joyykim 50f6f00
feat: Payment id타입 Long으로 변경, merchantUid UUID타입 컬럼 추가
Joyykim 625bb3a
Merge branch 'develop-server' into feature/iamport-cancel-payment-api
DWL5 3847a07
Merge branch 'develop-server' of https://github.com/woowacourse-teams…
Joyykim 2a674e2
Merge remote-tracking branch 'origin/feature/iamport-cancel-payment-a…
Joyykim 155311f
style: 리포매팅
Joyykim a9ad935
refactor: PaymentRequest -> PaymentCompleteRequest 리네임
Joyykim 8242c01
refactor: 사용되지 않는 클래스 제거
Joyykim 2638b42
refactor: 환불요청 dto의 merchantUid 타입 UUID로 변경
Joyykim f31567f
feat: Payment의 merchantUid 컬럼 nullable=false 추가
Joyykim 8e94396
feat: UUID 커스텀 Validator 생성, DTO의 merchantUid를 String타입으로 변경
Joyykim 05a161b
refactor: 스네이크케이스 -> 카멜케이스로 변경
Joyykim 49fcdac
feat: UUIDValidator null 검증 추가, 테스트작성
Joyykim File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
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
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.
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.
굿!! merchantUid는 null이 아니어야 하지