Skip to content

Add registration form, useRegister flow and toast - #11

Merged
TuvalZit18 merged 1 commit into
mainfrom
feat/registration
Oct 17, 2026
Merged

TuvalZit18 merged 1 commit into
mainfrom
feat/registration

Conversation

@TuvalZit18

Copy link
Copy Markdown
Owner
  • ToastProvider (MUI Snackbar + Alert) with useToast; mounted above the routes so the toast survives the redirect
  • registerUser API client; attribution is omitted when none is stored
  • useRegister: on success start the session, clear attribution, show a welcome toast and go to the validated returnTo; no retry, attribution kept on error
  • RegisterForm with client validation matching the API rules, field and general server errors, pending state
  • End-to-end tests: attribution sent and cleared, return to the promo modal, unsafe returnTo ignored, failure keeps attribution, 409, client validation

All three open questions are now decided (toast: MUI Snackbar with a small helper). Document the toast, the form and a known toast accessibility limit.

- ToastProvider (MUI Snackbar + Alert) with useToast; mounted above
  the routes so the toast survives the redirect
- registerUser API client; attribution is omitted when none is stored
- useRegister: on success start the session, clear attribution,
  show a welcome toast and go to the validated returnTo; no retry,
  attribution kept on error
- RegisterForm with client validation matching the API rules, field
  and general server errors, pending state
- End-to-end tests: attribution sent and cleared, return to the promo
  modal, unsafe returnTo ignored, failure keeps attribution, 409,
  client validation

All three open questions are now decided (toast: MUI Snackbar with a
small helper). Document the toast, the form and a known toast
accessibility limit.
@TuvalZit18
TuvalZit18 merged commit bfdec29 into main Oct 17, 2026
1 check passed
@TuvalZit18
TuvalZit18 deleted the feat/registration branch October 17, 2026 18:56
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