Conversation
additionally make the ticketmaster url a const variable reused in all buttons that redirect to tickets
There was a problem hiding this comment.
Pull request overview
Adds a dedicated “Bilety” (tickets) call-to-action in the mobile navbar sidebar and centralizes the Ticketmaster link into a shared config constant so ticket buttons don’t duplicate the URL.
Changes:
- Added
TICKETMASTER_URLconstant in shared config. - Replaced hard-coded Ticketmaster links in existing ticket buttons with
TICKETMASTER_URL. - Enabled/shipped a “Bilety” button in the mobile sidebar.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/config/data.ts | Introduces TICKETMASTER_URL constant for reuse across the app. |
| src/components/tickets-coming-soon.tsx | Uses TICKETMASTER_URL instead of an inline Ticketmaster link. |
| src/components/navbar.tsx | Uses TICKETMASTER_URL for the main navbar ticket button. |
| src/components/navbar-mobile.tsx | Adds a visible “Bilety” button to the mobile sidebar and points it to TICKETMASTER_URL. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <Button | ||
| as={Link} | ||
| href="https://docs.google.com/forms/d/e/1FAIpQLSel3CXdWuvrGTLkkrIMJWkvMqw6AiVz2cy2bAYw5bQntOFRQw/viewform?usp=send_form" | ||
| href={TICKETMASTER_URL} | ||
| className="!w-full !bg-gradient-main !py-4 before:!bg-black/10 [&_*]:hover:!text-white" | ||
| variant="default" | ||
| variantColor="white" | ||
| > | ||
| Turniej flanek | ||
| </Button> */} | ||
| Bilety | ||
| </Button> |
There was a problem hiding this comment.
The new Ticketmaster button in the mobile sheet doesn’t close the sidebar on click, unlike the NAV_LINKS entries above that call onOpenChange(false). Consider adding an onClick handler to the button/link to close the sheet for consistent behavior (especially if the navigation is interrupted or opened in a new tab).
additionally make the ticketmaster url a const variable reused in all buttons that redirect to tickets