-
Notifications
You must be signed in to change notification settings - Fork 0
feat(sets): multiple artists in sets view #8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| <span>{set.stages.name}</span> | ||
| </div> | ||
| )} | ||
| {formatTimeRange(set.time_start, set.time_end, use24Hour) && ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
convert to const
There was a problem hiding this 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 adds support for displaying multiple artists within a single set in the festival view, expanding the system from single-artist sets to multi-artist collaboration sets.
- Refactors set display components to handle both single and multi-artist scenarios
- Introduces new UI components for mixed artist images and multi-artist set information
- Creates a context-based architecture for shared set data and functionality
Reviewed Changes
Copilot reviewed 27 out of 27 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| src/pages/SetDetails.tsx | Updates set details page to conditionally render single vs multi-artist layouts |
| src/pages/EditionView.tsx | Removes unused isGrid prop and converts arrow function to function declaration |
| src/components/ui/card.tsx | Changes CardDescription from <p> to <div> element |
| src/components/SetDetail/*.tsx | New components for handling multi-artist set displays and individual artist cards |
| src/components/Index/shared/*.tsx | New shared components for social platform links, set metadata, and voting |
| src/components/Index/FestivalSetContext.tsx | New context provider for managing set-related state and functionality |
| src/components/Index/SetsPanel.tsx | Refactored to use new context-based architecture with conditional rendering |
No description provided.