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

fix: Match API 결제 상태 확인 로직 수정 #241

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

23tae
Copy link
Member

@23tae 23tae commented Dec 6, 2024

🔍️ 이 PR을 통해 해결하려는 문제가 무엇인가요?

어떤 기능을 구현한건지, 이슈 대응이라면 어떤 이슈인지 PR이 열리게 된 계기와 목적을 Reviewer 들이 쉽게 이해할 수 있도록 적어 주세요
일감 백로그 링크나 다이어그램, 피그마, 캡쳐를 첨부해도 좋아요

Match API 내 결제 상태 확인 로직의 오류로 인해 실제 결제가 완료된 사용자들이 신청 내역 조회시 미결제로 처리되는 문제가 발생했습니다.

주요 이슈:

  • 기존에는 MeetingTeam 엔티티의 payments 내 모든 payment가 success인 경우에만 결제 완료로 처리하고 있어, failed 기록이 하나라도 포함된 경우에는 미결제로 잘못 처리됨
  • 3대3 매치의 경우 세 명이 개별적으로 결제하는 방식으로 설계했으나, 실제로는 한 명이 일괄 결제하는 방식으로 운영되어 결제 여부가 잘못 처리됨

✨ 이 PR에서 핵심적으로 변경된 사항은 무엇일까요?

문제를 해결하면서 주요하게 변경된 사항들을 적어 주세요

결제 완료 상태 판단 로직을 다음과 같이 수정했습니다:

  • payments 배열 내 하나라도 success인 payment가 있으면 결제 완료로 처리하도록 변경
  • failed 상태의 payment 기록이 포함되어 있어도 success 건이 있다면 결제 완료로 처리

🔖 핵심 변경 사항 외에 추가적으로 변경된 부분이 있나요?

없으면 "없음" 이라고 기재해 주세요

  • 없음

🙏 Reviewer 분들이 이런 부분을 신경써서 봐 주시면 좋겠어요

개발 과정에서 다른 분들의 의견은 어떠한지 궁금했거나 크로스 체크가 필요하다고 느껴진 코드가 있다면 남겨주세요

🩺 이 PR에서 테스트 혹은 검증이 필요한 부분이 있을까요?

테스트가 필요한 항목이나 테스트 코드가 추가되었다면 함께 적어주세요

📌 PR 진행 시 이러한 점들을 참고해 주세요

  • Reviewer 분들은 코드 리뷰 시 좋은 코드의 방향을 제시하되, 코드 수정을 강제하지 말아 주세요.
  • Reviewer 분들은 좋은 코드를 발견한 경우, 칭찬과 격려를 아끼지 말아 주세요.
  • Review는 특수한 케이스가 아니면 Reviewer로 지정된 시점 기준으로 3일 이내에 진행해 주세요.
  • Comment 작성 시 Prefix로 P1, P2, P3 를 적어 주시면 Assignee가 보다 명확하게 Comment에 대해 대응할 수 있어요
    • P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
    • P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
    • P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)


📝 Assignee를 위한 CheckList

  • To-Do Item

- payments 전체가 SUCCESS인 경우만 결제 완료로 처리하던 로직을 수정
- 하나의 payment라도 SUCCESS인 경우 결제 완료로 처리하도록 변경
@23tae 23tae self-assigned this Dec 6, 2024
@23tae 23tae merged commit 0db5aca into main Dec 6, 2024
1 check failed
@23tae 23tae deleted the fix/matchapi-payment-check branch December 6, 2024 15:36
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.

1 participant