Skip to content

Add Staff Registration Link#127

Merged
Patle1234 merged 1 commit intomainfrom
staff-app-banner
Feb 23, 2026
Merged

Add Staff Registration Link#127
Patle1234 merged 1 commit intomainfrom
staff-app-banner

Conversation

@Patle1234
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings February 23, 2026 19:42
Copy link
Contributor

Copilot AI left a 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 a prominent yellow banner at the top of the site announcing that R|P 2026 staff applications are now open, linking to a Google Form for applications. The changes adapt the UI to accommodate this banner and update the landing page to focus on staff registration rather than general attendee registration.

Changes:

  • Added a yellow banner at the top of all pages linking to the staff application Google Form
  • Adjusted navbar positioning to account for the new banner (37px offset when not in flush mode)
  • Updated the landing page register button to point to staff applications and changed the button text to "REGISTER FOR STAFF"
  • Commented out the event date display (September 16-20, 2025) on the landing page

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
apps/site/src/App.tsx Added a yellow banner component announcing R
apps/site/src/components/Navbar.tsx Adjusted navbar top position from 0 to 37px when not in flush mode to accommodate the banner
apps/site/src/components/Home/Landing/Landing.tsx Updated register button to link to staff application form, changed button text, and commented out the 2025 event date

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@anandani4136 anandani4136 self-requested a review February 23, 2026 19:48
Copy link
Collaborator

@anandani4136 anandani4136 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Patle1234 Patle1234 merged commit b977b2a into main Feb 23, 2026
13 checks passed
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.

3 participants