Skip to content

feat: update application flow (#287) - #287

Open
jeessh wants to merge 2 commits into
mainfrom
jesse/improve-application-flow
Open

feat: update application flow (#287)#287
jeessh wants to merge 2 commits into
mainfrom
jesse/improve-application-flow

Conversation

@jeessh

@jeessh jeessh commented Jul 16, 2026

Copy link
Copy Markdown
Member

Implementation description

  • added a tooltip notification to warn the user of impending deadline (1 hour before, 5 minute grace period, end)

  • blocked submissions when the grace period ends, with a modal explaining the window has closed

  • kept the form alive through the grace period for anyone who started before the deadline, so refreshing no longer kicks them out (new visitors still get denied)

  • persisted in-progress answers to sessionStorage so a refresh keeps the form filled, which is cleared on successful submit

  • added a confirmation modal before submitting, since applications can't be edited afterwards

  • added an error message when a resume submission fails

  • ui changes (select dropdown, goofy left border, resume upload button, cursor pointers)

  • also ran linter and some other stray files got linted who didnt run their linters >:c

Steps to test

  1. change the application deadline to change states
  2. test that submissions work/get rejected depending on the state

What should reviewers focus on?

  • AppForm.tsx lowkey the most important one, rest are building blocks to it or small ui changes

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

@jeessh jeessh self-assigned this Jul 16, 2026
@netlify

netlify Bot commented Jul 16, 2026

Copy link
Copy Markdown

Deploy Preview for uwblueprint ready!

Name Link
🔨 Latest commit 9756f31
🔍 Latest deploy log https://app.netlify.com/projects/uwblueprint/deploys/6a595876bad2120008d67403
😎 Deploy Preview https://deploy-preview-287--uwblueprint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant