Skip to content
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

주문 히스토리 및 결제 히스토리 로직 수정 #89

Closed
wants to merge 3 commits into from

Conversation

pak0426
Copy link
Collaborator

@pak0426 pak0426 commented May 29, 2024

No description provided.

pak0426 added 3 commits May 24, 2024 18:26
- 관련 서비스 코드 및 테스트 코드 수정

issue: 86
- Payment 객체 수정, PaymentHistory 객체 생성
- Payment 생성 로직 수정 및 결제 완료 로직 추가

issue: 83
@pak0426 pak0426 requested a review from f-lab-odee May 29, 2024 12:01
@pak0426 pak0426 self-assigned this May 29, 2024
@f-lab-odee
Copy link
Collaborator

주문 히스토리 로직은 잘 수정하셨네요. 근데 결제쪽 로직은 아직 수정이 필요해보여서 아래 남겨주신 pr 수정되면 추가로 리뷰드릴게요 #85

@pak0426
Copy link
Collaborator Author

pak0426 commented Jun 12, 2024

위 브랜치는 feature#42에 병합했습니다. #85 링크에서 한번에 처리해서 머지하겠습니다.

@pak0426
Copy link
Collaborator Author

pak0426 commented Jun 12, 2024

위 코멘트를 이유로 PR을 닫습니다.

@pak0426 pak0426 closed this Jun 12, 2024
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.

2 participants