Conversation
There was a problem hiding this comment.
Pull request overview
Adds organizer-facing ticket management to the Event app, including Firestore plumbing, CRUD hooks/actions, and new UI routes/pages for listing and editing ticket types.
Changes:
- Introduce a
Ticketmodel, Firestore collection + converter, and security rules for/events/{eventId}/tickets. - Add data hooks (
useTickets,useTicket) and CRUD actions (create/update/delete). - Add Tickets UI (list, create, edit) plus menu entry and routing integration.
Reviewed changes
Copilot reviewed 20 out of 23 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
src/types.ts |
Adds the Ticket type used across services and UI. |
src/services/hooks/useTickets.ts |
Adds collection hook for event ticket types, ordered by startDate. |
src/services/hooks/useTicket.ts |
Adds single-ticket hook (currently unsafe when ticketId is null). |
src/services/firebase.ts |
Registers the tickets collection reference with a Firestore converter. |
src/services/converters.ts |
Adds ticketConverter for tickets collection reads/writes. |
src/events/page/tickets/components/TicketItem.tsx |
New ticket list item UI with edit + inline delete. |
src/events/page/tickets/components/TicketForm.tsx |
New create/edit form for tickets including currency selection. |
src/events/page/tickets/NewTicket.tsx |
New “create ticket” screen wiring form to Firestore mutation. |
src/events/page/tickets/EventTickets.tsx |
New “tickets list” screen rendering TicketItems. |
src/events/page/tickets/EventTicket.tsx |
New “edit ticket” screen wiring form to Firestore document mutation. |
src/events/page/EventScreenMenuItems.tsx |
Adds “Tickets” entry to the event sidebar menu. |
src/events/page/EventApp.tsx |
Adds /tickets, /tickets/new, /tickets/:id routes. |
src/events/new/NewEventDialog.tsx |
Creates a default “Standard” ticket when creating a new event. |
src/events/actions/updateTicket.ts |
Adds update action for a ticket doc. |
src/events/actions/deleteTicket.ts |
Adds delete action for a ticket doc. |
src/events/actions/createTicket.ts |
Adds create action for a ticket doc (currently type-mismatched). |
serviceapi-swagger/.empty |
Ensures the new hosting folder exists in the repo. |
functions/package-lock.json |
Adds license metadata entries for dependencies. |
firestore.rules |
Adds rules for reading/writing tickets subcollection under events. |
firebase.json |
Updates serviceapi hosting public directory to serviceapi-swagger. |
README.md |
Updates setup/deploy instructions and target naming guidance. |
.firebaserc.example |
Updates example project/targets naming to the new conventions. |
Files not reviewed (1)
- functions/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 20 out of 23 changed files in this pull request and generated 4 comments.
Files not reviewed (1)
- functions/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 20 out of 23 changed files in this pull request and generated 4 comments.
Files not reviewed (1)
- functions/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 20 out of 23 changed files in this pull request and generated 7 comments.
Files not reviewed (1)
- functions/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…orm and converters
…in EventTicket component
Add a complete ticket management system for events, allowing organizers to create, edit and delete ticket types.
Changes
Firestore infrastructure
Services
UI Pages