Skip to content

7.4-Booking-confirmation#73

Merged
RobTrb merged 13 commits intomainfrom
7.4-Booking-confirmation
Jun 6, 2025
Merged

7.4-Booking-confirmation#73
RobTrb merged 13 commits intomainfrom
7.4-Booking-confirmation

Conversation

@RobTrb
Copy link
Collaborator

@RobTrb RobTrb commented Jun 4, 2025

Pull Request Type

Please check the type of change your PR introduces:

  • Feature (a new feature for the project)

Description

Completed booking confirmation functionality in line with the previous booking pr. What has been added is that the client when booking a screening is taken to a finalized booking confirmation page containing the ticket and the options to print (only on desktop) or to download the ticket as pdf. Screenshots below:

Screenshots (if appropriate):

Finalized ticket (qr code contains the id of the booking):
image
Nav bar and buttons removed when printing for clarity:
image
Downloaded pdf in a4 format:
image

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have named this PR according to following naming convention: #[issue-number] PR description

@vercel
Copy link

vercel bot commented Jun 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
kino-next-js ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 6, 2025 0:57am

@RobTrb RobTrb linked an issue Jun 4, 2025 that may be closed by this pull request
3 tasks
Copy link
Collaborator

@Kalu32k Kalu32k left a comment

Choose a reason for hiding this comment

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

Looks very nice and works great, nice touch with print and pdf download feature!

- Missing { in a route
- Duplicated imports
- Missing mocks in booking.test
@RobTrb RobTrb merged commit 6d7b834 into main Jun 6, 2025
3 checks passed
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.

7.4 Booking confirmation

3 participants