Skip to content
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

feat: Displaying void message for applicants who failed to RSVP on time #289

Merged
merged 3 commits into from
Jan 21, 2024

Conversation

RandyHuynh5815
Copy link
Contributor

Resolves #281. Added display message and XCircle to applicants with the VOID status in the RSVP component as well as the two new statuses ACCEPTED and VOID. Verdict and submission components remain, but the waiver component is removed. Unsure if the language used for the display message would fit both scenarios mentioned well.

Copy link
Contributor

github-actions bot commented Jan 20, 2024

Deploy preview for irvinehacks-site-2024 ready!

Name IrvineHacks 2024 Site
Preview Visit Preview
Commit ef5b242

Copy link
Contributor

@samderanova samderanova left a comment

Choose a reason for hiding this comment

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

Thanks for the quick work on this! A few suggestions:

samderanova
samderanova previously approved these changes Jan 20, 2024
Copy link
Contributor

@samderanova samderanova left a comment

Choose a reason for hiding this comment

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

Thanks for getting this done! Please squash merge when you get two approvals.

waalbert
waalbert previously approved these changes Jan 21, 2024
Copy link
Contributor

@samderanova samderanova left a comment

Choose a reason for hiding this comment

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

Not sure if you saw a merge conflict regarding the Button component, but currently the RSVP button causes horizontal overflow on mobile views. See Button.tsx from #283.

@RandyHuynh5815 RandyHuynh5815 requested review from a team and removed request for taesungh January 21, 2024 01:41
Copy link
Contributor

@samderanova samderanova left a comment

Choose a reason for hiding this comment

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

My mistake, seems like I didn't pull the changes properly. Looks good!

@RandyHuynh5815 RandyHuynh5815 merged commit 84839bb into main Jan 21, 2024
4 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.

[Portal] Display void message for applicants who failed to RSVP on time
3 participants