Skip to content

Feat: 좌석예약 API 연동 (작업중)#46

Merged
zaewc merged 16 commits intodevelopfrom
feat/booking-api
Aug 14, 2025
Merged

Feat: 좌석예약 API 연동 (작업중)#46
zaewc merged 16 commits intodevelopfrom
feat/booking-api

Conversation

@zaewc
Copy link
Copy Markdown
Member

@zaewc zaewc commented Aug 13, 2025

💡 PR 요약

좌석예약 API 연동 (작업중)

@github-actions
Copy link
Copy Markdown

✅ CI 성공

@github-actions
Copy link
Copy Markdown

✅ CI 성공

@zaewc zaewc marked this pull request as draft August 13, 2025 10:38
@bae080311 bae080311 requested review from bae080311 and Copilot August 13, 2025 11:51
@bae080311 bae080311 self-assigned this Aug 13, 2025
@bae080311 bae080311 added the enhancement New feature or request label Aug 13, 2025
@bae080311 bae080311 removed their assignment Aug 13, 2025
@zaewc zaewc self-assigned this Aug 13, 2025

This comment was marked as outdated.

@github-actions
Copy link
Copy Markdown

✅ CI 성공

@bae080311 bae080311 requested a review from Copilot August 14, 2025 00:54
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR integrates seat reservation API functionality to enable real-time seat status updates and booking operations. The changes focus on replacing static seat data with dynamic API-driven seat state management.

Key changes:

  • Added seat state management with React Query for real-time seat availability
  • Implemented seat booking API integration with mutation handling
  • Restructured seat selection logic to work with dynamic seat data

Reviewed Changes

Copilot reviewed 14 out of 16 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
src/widgets/main/PreliminaryFourthSection/index.tsx Minor formatting changes to HTML line breaks
src/widgets/booking/ui/SelectSection/index.tsx Added seat state prefetching and dynamic seat count display
src/widgets/booking/ui/SeatSection/index.tsx Integrated dynamic seat loading with fallback handling
src/widgets/booking/lib/useSeatState.ts New hook for managing seat state queries and prefetching
src/widgets/booking/lib/useSeatSelection.ts Fixed import path for types
src/widgets/booking/lib/useSeatBooking.ts New hook for seat booking mutations
src/views/booking/ui/BookingPage/index.tsx Integrated booking mutation with loading states
src/entities/booking/ui/SeatItem/index.tsx Code formatting improvements
src/entities/booking/ui/SeatGrid/index.tsx Added dynamic seat data integration with cached queries
src/entities/booking/model/types.ts Added API response types and helper functions
src/entities/booking/model/seatLayouts.ts Removed unused properties from SeatLayout interface
src/entities/booking/api/seatBooking.ts New API function for seat booking
src/entities/booking/api/getSeatState.ts New API function for fetching seat states
src/entities/booking/api/cancelSeatBooking.ts New API function for canceling seat bookings

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/widgets/booking/lib/useSeatState.ts
Comment thread src/widgets/booking/lib/useSeatState.ts Outdated
Comment thread src/widgets/booking/ui/SeatSection/index.tsx
Comment thread src/entities/booking/model/types.ts
Comment thread src/widgets/booking/ui/SelectSection/index.tsx
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

✅ CI 성공

@zaewc zaewc force-pushed the feat/booking-api branch from 7b44547 to bf170f9 Compare August 14, 2025 00:58
@github-actions
Copy link
Copy Markdown

✅ CI 성공

@zaewc zaewc marked this pull request as ready for review August 14, 2025 01:00
@github-actions
Copy link
Copy Markdown

✅ CI 성공

@github-actions
Copy link
Copy Markdown

✅ CI 성공

Copy link
Copy Markdown
Member

@bae080311 bae080311 left a comment

Choose a reason for hiding this comment

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

수고 햇어용

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

에러 핸들링 하나염?

toast.success("좌석 예매가 완료되었습니다.");
},
onError: (error) => {
console.error(error);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

그냥 콘솔 빼는거 어때염

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

API 배포되고 테스트 잘 되면 뺄게용

@github-actions
Copy link
Copy Markdown

✅ CI 성공

@github-actions
Copy link
Copy Markdown

✅ CI 성공

@zaewc zaewc merged commit 4a8f987 into develop Aug 14, 2025
2 checks passed
@bae080311 bae080311 deleted the feat/booking-api branch August 14, 2025 05:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants