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

Yeonwoo/payments PR 10/10 #154

Closed
wants to merge 7 commits into from
Closed

Yeonwoo/payments PR 10/10 #154

wants to merge 7 commits into from

Conversation

Yoo117
Copy link
Contributor

@Yoo117 Yoo117 commented Oct 9, 2024

기존 내용에서 변경점

  • kakaopay 요청/승인/취소 함수를 services.py로 분리했습니다. paymentview의 내용이 좀 더 명시적으로 바뀌었습니다.
  • orderview에서 장바구니에서 상품을 전환하는 것 뿐만이 아니라, 바로 주문을 생성하는 기능을 추가했습니다.
  • 예외 처리 로직을 조금 수정했습니다. paymentview에서 이제 좀 더 명확하게 에러를 표시하며, 예외처리를 좀 더 확실하게 진행했습니다.

추가된 내용

  • services 파일을 추가했습니다. kakaopay 요청/승인/취소(환불) 기능을 분리하였습니다.
  • test 파일을 추가했습니다. 현재 모든 테스트를 통과한 상태입니다.

@Yoo117 Yoo117 closed this Oct 9, 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.

1 participant