Skip to content

Fix scenario page error rendering - #144

Open
jonny5 wants to merge 4 commits into
mainfrom
jk-fix-flash-ui
Open

Fix scenario page error rendering#144
jonny5 wants to merge 4 commits into
mainfrom
jk-fix-flash-ui

Conversation

@jonny5

@jonny5 jonny5 commented Aug 30, 2026

Copy link
Copy Markdown
Collaborator

See #114 for existing behavior that this attempts to resolve.

  • Reverts previous commit which used javascript for this, rather than server side validation
  • No longer rely on flash messages for form errors
  • Render errors inline for more intuitive user feedback
output3.mp4

jonny5 added 4 commits August 30, 2026 10:07
…ocation that is higher than the allowable total giving amount (#128)"

This reverts commit 5d1582f.
The form lives in a top-layer <dialog>, so a flash rendered behind the
modal and the redirect dropped the user's input.
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.

2 participants