Skip to content

Conversation

@chiptus
Copy link
Owner

@chiptus chiptus commented Sep 5, 2025

No description provided.

chiptus and others added 3 commits September 5, 2025 14:20
- Add slug columns to groups and stages tables with proper uniqueness constraints
- Update admin routes from festivalId/editionId to festivalSlug/editionSlug
- Convert all admin components to use slug-based navigation and queries
- Fix seed file to include slug values for groups and stages
- Update create mutations to auto-generate slugs for new entities
- Add proper loading states and TypeScript safety for slug-based queries
- Maintain React hooks rules compliance with proper hook ordering

Admin URLs now use human-readable slugs:
- /admin/festivals/boom-festival instead of /admin/festivals/uuid
- /admin/festivals/boom-festival/editions/2025 instead of /admin/festivals/uuid/editions/uuid

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Replace duplicate API calls in StageManagement and SetManagement
- Use React Router's useOutletContext to pass edition data from parent
- Eliminate loading states in child components since data comes from parent
- Remove redundant slug-to-ID resolution in child components
- Simplify component props by removing editionSlug/festivalSlug parameters

Benefits:
- Faster page loads (no redundant API calls)
- Cleaner component architecture
- Better user experience with fewer loading states
- More efficient data flow

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@vercel
Copy link

vercel bot commented Sep 5, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
upline Ready Ready Preview Comment Sep 5, 2025 0:18am

@chiptus chiptus merged commit 902e932 into main Sep 5, 2025
7 checks passed
@chiptus chiptus deleted the feat/slug-urls branch September 5, 2025 12:28
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.

2 participants