Skip to content

[BE] 티켓 완료 시각 추가.#186

Merged
shininghyunho merged 1 commit intodevfrom
backend-185
Feb 4, 2026
Merged

[BE] 티켓 완료 시각 추가.#186
shininghyunho merged 1 commit intodevfrom
backend-185

Conversation

@shininghyunho
Copy link
Copy Markdown
Collaborator

@shininghyunho shininghyunho commented Feb 4, 2026

  • CreateReservationResponseDto: reserved_at 필드 추가
  • ReservationService: 예약 성공 시 현재 시각(ISO 8601) 캡처 로직 추가

Summary by CodeRabbit

새로운 기능

  • 예약 응답에 예약 완료 시간이 ISO 8601 형식으로 추가되었습니다. 이제 API 응답에서 예약이 정확히 완료된 시점을 확인할 수 있습니다.

- CreateReservationResponseDto: reserved_at 필드 추가
- ReservationService: 예약 성공 시 현재 시각(ISO 8601) 캡처 로직 추가
- 테스트 코드: reserved_at 필드 검증 로직 반영
@shininghyunho shininghyunho self-assigned this Feb 4, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 4, 2026

📝 Walkthrough

Walkthrough

예약 응답 DTO에 ISO 8601 형식의 예약 완료 시간을 나타내는 reserved_at 필드를 추가했습니다. ReservationService의 reserve() 메서드에서 이 필드를 계산하여 반환하고, 테스트에서 이 필드의 존재와 타입을 검증합니다.

Changes

Cohort / File(s) Summary
DTO 정의
backend/ticket-server/src/reservation/dto/create-reservation-response.dto.ts
CreateReservationResponseDto에 reserved_at: string 필드와 ApiProperty 어노테이션을 추가하여 ISO 8601 완료 시간을 표현합니다.
서비스 로직 및 테스트
backend/ticket-server/src/reservation/reservation.service.ts, backend/ticket-server/src/reservation/reservation.service.spec.ts
reserve() 메서드에서 완료 후 ISO 문자열 형식의 타임스탬프를 계산하여 반환 객체에 추가하고, 테스트에서 reserved_at 필드의 정의 여부와 문자열 타입을 검증합니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

  • web10-beastcamp#177: 동일한 CreateReservationResponseDto와 ReservationService.reserve()를 수정하며, 응답 필드를 추가하는 점에서 관련되어 있습니다 (본 PR은 reserved_at, 해당 PR은 virtual_user_size 추가).
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning PR 설명은 두 가지 주요 변경사항을 간단히 나열했으나, 제공된 템플릿의 필수 섹션(요약, 연결 이슈, 주요 고민과 해결방법, 리뷰 포인트 등)을 충족하지 않습니다. 템플릿에 따라 🧭 Summary, 🔗 Linked Issue, 🛠 개발 기능, 🧩 주요 고민과 해결 방법, 🔍 리뷰 포인트 섹션을 추가하여 상세히 작성해주세요.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 예약 완료 시각 추가라는 주요 변경 사항을 명확하고 간결하게 요약하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch backend-185

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.

@shininghyunho shininghyunho merged commit 3b5ae83 into dev Feb 4, 2026
5 checks passed
@viixix viixix deleted the backend-185 branch February 5, 2026 05:17
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