Fix: Reopening ideathon registrations - #60
Merged
Conversation
|
@Akshat-Raj is attempting to deploy a commit to the Genesis' projects Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Pull request overview
Reopens Ideathon registrations by re-enabling it in the registration UI and allowing API registrations again, while keeping Clash Royale closed and updating the hero banner copy accordingly.
Changes:
- Re-enabled the Ideathon (
pitch_perfect) card on the registration page. - Updated the registration API to only block
clash_royale(allowing Ideathon registrations again). - Adjusted the Hero banner text to reflect that only Clash Royale registrations are closed.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/components/sections/Hero.tsx | Updates hero banner message to no longer state Ideathon is closed. |
| src/app/register/page.tsx | Re-adds Ideathon to the list of selectable registration events. |
| src/app/api/register/route.ts | Removes the API-side block on Ideathon registrations; keeps Clash Royale blocked. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| leading-relaxed md:leading-relaxed" | ||
| > | ||
| <span className="font-black text-[15px] sm:text-base md:text-xl">CLASH ROYALE AND IDEATHON REGISTRATIONS ARE NOW CLOSED!</span> <span className="font-semibold">Please register for other exciting events.</span> <span className="text-[#22c55e] dark:text-[#4ade80] font-bold">On-spot registrations will be available on the day of the fest!</span> | ||
| <span className="font-black text-[15px] sm:text-base md:text-xl">CLASH ROYALE REGISTRATIONS ARE NOW CLOSED!</span> <span className="font-semibold">Please register for other exciting events.</span><br></br> <span className="text-[#22c55e] dark:text-[#4ade80] font-bold">On-spot registrations will be available on the day of the fest!</span> |
There was a problem hiding this comment.
<br></br> is a void element; in JSX this should be self-closing (<br />). The codebase uses the self-closing form elsewhere (e.g., src/app/about/page.tsx), and this avoids React/ESLint void-element warnings.
Suggested change
| <span className="font-black text-[15px] sm:text-base md:text-xl">CLASH ROYALE REGISTRATIONS ARE NOW CLOSED!</span> <span className="font-semibold">Please register for other exciting events.</span><br></br> <span className="text-[#22c55e] dark:text-[#4ade80] font-bold">On-spot registrations will be available on the day of the fest!</span> | |
| <span className="font-black text-[15px] sm:text-base md:text-xl">CLASH ROYALE REGISTRATIONS ARE NOW CLOSED!</span> <span className="font-semibold">Please register for other exciting events.</span><br /> <span className="text-[#22c55e] dark:text-[#4ade80] font-bold">On-spot registrations will be available on the day of the fest!</span> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix: Reopening ideathon registrations