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

[feat] TossPayment 결제 로직 코틀린 마이그레이션 #11

Merged
merged 7 commits into from
Jan 27, 2025

Conversation

ohdeng02
Copy link
Collaborator

@ohdeng02 ohdeng02 commented Jan 24, 2025

closed #3

📌 과제 설명

토스페이먼츠 결제 로직 코틀린으로 마이그레이션

👩‍💻 요구 사항과 구현 내용

  • 결제 예외처리 구현
  • Order관련 주문 로직 생성(임시적으로 생성한 로직이라 민규님 작성하는 로직으로 수정될 예정입니다)
  • ParkingStatus 엔티티 parking컬럼 private 제거
  • TossPaymentService, TossPaymentController 결제 로직 구현

✅ 피드백 반영사항

코드리뷰 남겨주시면 감사하겠습니다!

✅ PR 포인트 & 궁금한 점

추후 개선할 사항

  • 결제로직중 세션 정보 쿠키로 frontend 전송 로직 구현
  • enum클래스 converter로 리팩토링
  • Entity 주생성자 null 허용 파라미터 다시 검토
  • setter 접근제한자 수정

특이사항

현재 결제 로직이 API자체로는 동작하나, 프론트와 연동시 추가적인 구현이 필요한 상황입니다.

  1. 결제 중 결제금액을 서버 세션에 저장하여 검증하는 로직
  2. 결제 중간에 '결제 승인하기'요청 화면 추가(결제 인증 후 결제 완료 페이지로 넘어가기 전)
    d
  • 위 두가지에 대해 구현해둔 코드가 어느정도 있어서(프론트엔드,백엔드) 도현님과 상의해보거나, 결제 쪽 프론트 코드 올려주시면 반영하겠습니다

@ohdeng02 ohdeng02 self-assigned this Jan 24, 2025
@ohdeng02 ohdeng02 linked an issue Jan 24, 2025 that may be closed by this pull request
4 tasks
@citycozy citycozy merged commit 1049767 into develop Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat]: 주문 & Tosspayments 결제 구현
2 participants