Skip to content

Conversation

@Adr1an04
Copy link
Collaborator

@Adr1an04 Adr1an04 commented Jan 3, 2026

Why

fixes after merge, time picker wasnt defaulting AM properly in the backend, card borders were getting clipped, and needed redirect

What

  • time picker am and pm fix - it now correctly defaults to am in the backend (was showing am visually but not allowing a submission)
  • card border clipping fix
  • redirect after submit - success screen for 5 seconds then redirects to blade
  • form not found modal - let team member know if form doesnt exist
  • uses blade member name instead of discord name

Test Plan

  • tested time picker by filling hours/minutes without touching am/pm
  • tested redirect countdown shows "Redirecting in 5..." and goes to landing page after submit
  • tested form not found shows modal with message to contact team member

Pictures

Screenshot 2026-01-03 at 2 59 23 PM Screenshot 2026-01-03 at 2 57 22 PM Screenshot 2026-01-03 at 2 56 36 PM

Copy link
Contributor

@DVidal1205 DVidal1205 left a comment

Choose a reason for hiding this comment

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

my goat

@DVidal1205 DVidal1205 added this pull request to the merge queue Jan 3, 2026
Merged via the queue into main with commit 9e82586 Jan 3, 2026
5 checks passed
@DVidal1205 DVidal1205 deleted the fix-form-responder-name branch January 3, 2026 20:16
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