Phase 6: public UX — form validation, confirmation screen, verify states, self-signup - #20
Closed
ArneeMe wants to merge 1 commit into
Closed
Phase 6: public UX — form validation, confirmation screen, verify states, self-signup#20ArneeMe wants to merge 1 commit into
ArneeMe wants to merge 1 commit into
Conversation
- SchemaForm now validates before submit: required-by-default per the form schema (respects optional), date/number sanity, inline errors that clear as the volunteer fixes them. Pure logic extracted to util/validateFormField.ts with unit tests (vitest has no JSX setup, matching the repo's pure-module test pattern). - Successful submission shows a real confirmation screen (what happens next, that data is auto-deleted at issuance) instead of a dialog over the stale form. - Verify page: unmistakable green/red status with an explanation of what invalid can mean, and the field grid reframed as 'attesterte opplysninger' you can perturb to see verification fail. - /registrer self-signup via nhost signUpEmailPassword. A fresh account has zero org memberships so it grants nothing by itself; linked from the login page and the Medlemmer helper text. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BFHkXUuHJME4cF9AJWbavB
Deploying inf319-certificate with
|
| Latest commit: |
669d229
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://9a4a8325.inf319-certificate.pages.dev |
| Branch Preview URL: | https://claude-phase-6-public-ux.inf319-certificate.pages.dev |
This was referenced Jul 11, 2026
Owner
Author
|
Superseded by the review-ordered stack #25 → #29 (form validation/confirmation/verify UX in #27, signup in #29). Closing. Generated by Claude Code |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #17 (base is
claude/phase-3-onboardingso the diff shows only phase 6 — the signup flow builds on phase 3's members page). Merge order: #16 → #17 → this, retargeting bases as each lands. Independent of the #18 → #19 stack.Volunteer form
optional), date and number sanity checks, inline error messages that clear as the volunteer types. Pure logic lives insrc/util/validateFormField.tswith unit tests. Notably this fixes required dropdowns, which MUI never enforced natively.Verify page
Self-signup
/registrervia NhostsignUpEmailPassword. A fresh account carries zero org memberships, so it grants no access by itself — an existing member must still add the address on the Medlemmer page. Linked from the login page and the members-page helper text. Handles both Nhost modes (immediate session vs. email verification required).Verification
/registrer,/login, and the verify page all render the new states.🤖 Generated with Claude Code
https://claude.ai/code/session_01BFHkXUuHJME4cF9AJWbavB
Generated by Claude Code