week10 review #144
Closed
week10 review #144
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
MemberService <-> SecurityBeanGenerator 순환 참조 문제 해결
- 다중 결제취소 등등을 위해 entity 구조를 변경했습니다 - AbstractOrder를 상속받는 productOrder, farmOrder - 각 Order Entity는 list의 orderedProduct, orderedFarm을 가짐 - 각 Order는 Payment 방식을 기록한 Payment entity를 가짐 - 각 Order는 order의 detail을 가지는 orderdetail을 가짐 - orderedProduct는 aftersales를 list로 가짐 - aftersales는 refunddetail을 1:1 관계로 가짐
- IamportResponseException을 전역적으로 잡기 위해 advice를 추가하였습니다 - IOException을 전역적으로 잡기 위해 advice를 추가하였습니다
- 기존 paymentService가 너무 많은 결제 로직을 담당하였습니다 근데 다론 service에서도 이를 호출하니 너무 복잡한 의존관계가 된 것 같아서 paymentUtil을 구현하였습니다.
- 두 개의 도메인에 대해서 service를 나누었습니다 두 service는 나중에 인터페이스를 상속받아서 구현하겠습니다
- 두 도메인에서 공통적으로 사용하는 cancel을 제네릭을 사용하여 인터페이스를 작성하였습니다
- record 객체로 구현하였습니다
- 상세 내용은 내일 작성하겠습니다
…e/ISSUE-86 # Conflicts: # src/main/java/poomasi/domain/member/entity/Member.java # src/main/java/poomasi/domain/product/entity/Product.java # src/main/java/poomasi/global/error/BusinessError.java
…E-86 결제 order entity 수정 및 결제 구현
…e/ISSUE-111 # Conflicts: # src/main/java/poomasi/global/error/BusinessError.java
…E-111 [issue-111] 결제 처리 및 사후 관련 구현
…E-111 111번 다시 merge
Merge pull request #145 from kakao-tech-campus-2nd-step3/feature/ISSU…
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
해결하려는 문제가 무엇인가요?
어떻게 해결했나요?
코드 리뷰시 요청 사항
더 하고 싶은 말