Skip to content

Conversation

@munyeong0103
Copy link
Contributor

📝요약(Summary)

이슈 번호 : #167

🔨변경 사항(Changes)

  • [order service] 기존 주문 생성 로직 순서 변경, 트랜잭션 분리
  • [payment service] order 와의 의존성 분리
  • payment facade 생성 -> 트랜잭션 관리

😉리뷰 요구사항

  • rollback 로직이 미적용된 코드입니다. 감안해서 테스트 부탁드리겠습니다
  • 로직에 오류가 없는지 확인 부탁드리겠습니다

Copy link
Contributor

@jeemin65-pixel jeemin65-pixel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아 문영님 paymentServiceTest 에서 기대값이 PAID 로 되어 있어서 CI test 실패한 것 같습니다 확인 부탁드립니다!

@munyeong0103
Copy link
Contributor Author

아 문영님 paymentServiceTest 에서 기대값이 PAID 로 되어 있어서 CI test 실패한 것 같습니다 확인 부탁드립니다!

앗 수정해서 다시 올렸습니다 ! 확인 감사합니다 :)

Copy link
Contributor

@jong15325 jong15325 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

코드 전반적으로 확인했습니다, 자세한건 테스트를 진행해봐야 할 것 같습니다

Copy link
Contributor

@DaveLee-b DaveLee-b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다! 전반적으로 큰 문제는 없는것 같습니다!

@munyeong0103 munyeong0103 merged commit 5a311f3 into develop Jan 6, 2026
1 check passed
@munyeong0103 munyeong0103 deleted the SHOP-133 branch January 6, 2026 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants