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 #172

Merged
merged 37 commits into from
Oct 12, 2024
Merged

Yeonwoo/payments #172

merged 37 commits into from
Oct 12, 2024

Conversation

Yoo117
Copy link
Contributor

@Yoo117 Yoo117 commented Oct 12, 2024

기존에서 변경점

  • view의 내용을 mixin으로 일부 분리했습니다. 공통적으로 사용하는 기능들이랑, 에러처리 로직을 넣었습니다.
  • view에서 orderview와 paymentview에서 특정 주문을 대상으로 하는 post를 수정했습니다. 이제 id를 찾고 지정해서 적용하는 것이 아닌, 무조건 order_status=pending 상태의 주문만 조회하며, 결제를 처리합니다. 사용자의 사용성에 초점을 맞췄습니다.
  • test의 내용을 추가했습니다. 결제 처리 관련쪽의 내용을 좀 더 늘렸습니다.

새로 추가된 내용

  • admin을 추가했습니다. 기본적인 정보들을 관리할 수 있습니다.
  • mixin을 추가했습니다. view에서의 공통기능 부분을 분리한 형태이며, service를 참조하여 기능을 수행합니다.
  • permission을 추가하여 좀 더 보안을 강화했습니다.

현재 materials에서 migration 미추가로 test가 전부 error가 나고 있습니다. migration 추가 후 migrate를 하면 테스트는 전부 통과가 됩니다.

Yoo117 added 30 commits October 10, 2024 16:16
@AlbertImKr AlbertImKr merged commit f2fa4aa into dev Oct 12, 2024
1 check failed
@AlbertImKr AlbertImKr deleted the yeonwoo/payments branch October 12, 2024 15:22
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