Skip to content

[FEATURE] Payment 관련 Reservation 서비스 메서드 추가#213

Merged
friox merged 9 commits into
developfrom
feature/203-payment-reservation-method
Aug 5, 2025
Merged

[FEATURE] Payment 관련 Reservation 서비스 메서드 추가#213
friox merged 9 commits into
developfrom
feature/203-payment-reservation-method

Conversation

@friox

@friox friox commented Aug 5, 2025

Copy link
Copy Markdown
Contributor

관련 Issue (필수)

주요 변경 사항 (필수)

  • Ticket - QR 엔티티 간 CASCADE 설정 추가
  • Reservation 인스턴스를 파라미터로 사용하는 Reservation 관련 항목 삭제 메서드 구현

리뷰어 참고 사항

Warning

해당 PR을 반영하여 작업한 후 cherry-pick 으로 작업한 커밋을 분리한 브랜치입니다.
이 브랜치(PR) 만으로는 위 PR이 머지되기 전까지 테스트가 불가합니다. (빌드도 불가능합니다.)
위 PR이 머지된 후 rebase 하겠습니다.

(추가) rebase 완료했습니다, 테스트 가능합니다.

임의의 엔드포인트를 생성하여 테스트 완료했습니다.
예약 생성 후 생성 된 예약을 이용하여 메서드를 테스트 한 결과,

  • Reservation 인스턴스는 CANCELLED 상태로 변경되었습니다.
  • SeatReservation 인스턴스가 삭제되었습니다.
  • Ticket, QR 인스턴스가 삭제되었습니다.

추후 테스트 시 Ticket, QR 삭제에 문제가 발생하신다면,
기존 두 테이블을 DROP 한 후 진행하시면 되겠습니다.

추가 정보

없음

PR 작성 체크리스트 (필수)

  • 제목이 Issue와 동일함을 확인했습니다.
  • 리뷰어를 지정했습니다.
  • 프로젝트를 연결했습니다.

@EndlessMilkyway EndlessMilkyway left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

해당 부분을 신속하게 작업해주셔서 결제 취소 로직을 빠르게 구현할 수 있을 것 같습니다. 고생많으셨습니다!

@friox
friox force-pushed the feature/203-payment-reservation-method branch from fa74c75 to a366dfd Compare August 5, 2025 04:03

@chanwonlee chanwonlee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Payment 관련해서 작업 빠르게 해주셔서 감사합니다! 고생하셨습니다!

Comment thread src/main/java/com/sudo/railo/booking/application/SeatReservationService.java Outdated
Comment thread src/main/java/com/sudo/railo/booking/application/TicketService.java
@friox
friox force-pushed the feature/203-payment-reservation-method branch from a366dfd to 5e2c471 Compare August 5, 2025 04:32
@friox
friox force-pushed the feature/203-payment-reservation-method branch from 078956f to f31f139 Compare August 5, 2025 04:44

@Yunsung-Jo Yunsung-Jo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다! 👍

@Ogu1208 Ogu1208 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다~ 늦게 확인했네요 ㅠㅠ
고생하셨습니다!! 👍👍

@friox
friox merged commit 873f744 into develop Aug 5, 2025
3 checks passed
@friox
friox deleted the feature/203-payment-reservation-method branch August 5, 2025 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[FEATURE] Payment 관련 Reservation 서비스 메서드 추가

5 participants