-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: 결제 환불 로직 리팩토링 #245
feat: 결제 환불 로직 리팩토링 #245
Conversation
- try - catch 시 refreshToken 추가
- 학적 상태 졸업생, 대학원생으로 변경 시 학부 학번 삭제 - 전화번호 중복 예외 출력
- /matches/{matchId}/result -> /matches/result로 변경 - 요청 파라미터를 teamId에서 teamType으로 변경
- 파라미터 teamType으로 변경 - 캐시 키 형식 변경
- 책임 분리를 위해 새로운 DTO 생성 - ArrayList 변환 로직에서 preference 변환 제거
…life/server-meeting into refactor/match-api-endpoints
… refactor/match-api-endpoints
… refactor/match-api-endpoints
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.
확인했습니다👍
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.
로직은 잘 구성한 것 같습니다 고생하셨습니다
결제해놓고 결과 발표 전 중도 취소하는 케이스에 대해서는 이미 상태값 변화가 되어있는건가요?
(아니면 그런 케이스가 없었나요? 이전에는 있었어서 물어봅니다)
네넵 취소는 이미 됐습니다 |
🔍️ 이 PR을 통해 해결하려는 문제가 무엇인가요?
✨ 이 PR에서 핵심적으로 변경된 사항은 무엇일까요?
🔖 핵심 변경 사항 외에 추가적으로 변경된 부분이 있나요?
🙏 Reviewer 분들이 이런 부분을 신경써서 봐 주시면 좋겠어요
🩺 이 PR에서 테스트 혹은 검증이 필요한 부분이 있을까요?
📌 PR 진행 시 이러한 점들을 참고해 주세요
📝 Assignee를 위한 CheckList