Skip to content

feat: 구매한 티켓 정보 조회 api 필드 추가 - #222#223

Merged
sjk4618 merged 2 commits into
devfrom
feat/#222
Jan 18, 2026
Merged

feat: 구매한 티켓 정보 조회 api 필드 추가 - #222#223
sjk4618 merged 2 commits into
devfrom
feat/#222

Conversation

@sjk4618
Copy link
Copy Markdown
Member

@sjk4618 sjk4618 commented Jan 18, 2026

🔥Pull requests

⛳️ 작업한 브랜치

👷 작업한 내용

  • feat: 구매한 티켓 정보 조회 api - paymentPrice 필드 추가

🚨 참고 사항

@sjk4618 sjk4618 self-assigned this Jan 18, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 18, 2026

📝 Walkthrough

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 티켓 구매 정보 조회 시 결제 금액이 표시됩니다.
    • 환불 금액과 함께 결제 금액 정보가 통합되어 제공됩니다.
  • 개선 사항

    • 내부 디버그 로그 제거로 시스템 성능을 최적화했습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

구매 티켓 정보 조회 API 응답에 결제 금액 필드를 추가하는 변경사항입니다. Order 데이터 모델에 paymentPrice 필드를 추가하고, PaymentRetriever를 통해 결제 금액을 조회하여 DTO에 전달하는 로직을 구현했습니다. 또한 JwtAuthenticationFilter의 디버그 로깅을 제거했습니다.

Changes

Cohort / File(s) 변경 요약
결제 금액 필드 추가
src/main/java/com/permitseoul/permitserver/domain/ticket/api/dto/res/UserBuyTicketInfoResponse.java
Order 레코드에 String paymentPrice 필드 추가; 생성자 서명 업데이트
티켓 서비스 결제 금액 통합
src/main/java/com/permitseoul/permitserver/domain/ticket/api/service/TicketService.java
findPaymentAmountsByOrderId() 헬퍼 메서드 추가; getUserBuyTicketInfo()에서 결제 금액 조회; convertToOrderList() 서명 업데이트로 paymentAmountByOrderId 파라미터 추가; 주문 생성 시 포맷된 결제 금액 포함
인증 필터 정리
src/main/java/com/permitseoul/permitserver/global/filter/JwtAuthenticationFilter.java
요청 헤더 및 쿠키 디버그 로깅 제거

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning JwtAuthenticationFilter에서 테스팅 로그 제거는 부수적인 정리 작업이나, 이는 PR의 주요 목표(paymentPrice 필드 추가)와는 직접적인 관련이 없는 변경입니다. JwtAuthenticationFilter의 로그 제거는 별도의 PR로 분리하거나, 메인 작업과의 관계를 PR 설명에 명시하는 것을 권장합니다.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경 사항의 주요 내용과 정확히 일치합니다. 구매한 티켓 정보 조회 API에 paymentPrice 필드를 추가하는 작업을 명확하게 반영하고 있으며, 이슈 번호(#222)도 포함되어 있습니다.
Description check ✅ Passed PR 설명이 변경 사항과 관련이 있습니다. 작업 브랜치(Resolved: #222)와 작업 내용(paymentPrice 필드 추가)이 명시되어 있어 변경 사항을 이해하는 데 충분합니다.
Linked Issues check ✅ Passed 코드 변경사항이 링크된 이슈 #222의 목표를 충족합니다. UserBuyTicketInfoResponse에 paymentPrice 필드가 추가되었으며, TicketService에서 결제 금액 데이터를 조회하여 전달하는 로직이 구현되었습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sjk4618 sjk4618 merged commit 975a515 into dev Jan 18, 2026
2 checks passed
@sjk4618 sjk4618 deleted the feat/#222 branch January 18, 2026 15:33
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.

feat: 구매한 티켓 정보 조회 api 필드 추가

1 participant