Skip to content

fix(web): make signup resumable after incomplete account creation#14530

Merged
raymondjacobson merged 2 commits into
mainfrom
codex/resumable-signup
Jul 20, 2026
Merged

fix(web): make signup resumable after incomplete account creation#14530
raymondjacobson merged 2 commits into
mainfrom
codex/resumable-signup

Conversation

@raymondjacobson

Copy link
Copy Markdown
Member

Summary

  • checkpoint successful Identity registration so a failed core write can retry without registering the email again
  • resume Identity-only accounts at handle/profile setup in the current URL-based signup router
  • reuse an indexed user after an uncertain relay result and update partially completed users instead of creating a second user ID
  • keep create-user failures retryable, report them, and prevent the confirmer from overwriting failure with success
  • replace the indefinite loading state with a Try Again action

Root cause

Signup commits Identity credentials before submitting the core CreateUser write. When the chain or relay is unavailable, Identity reserves the email but the core user is never created. The existing incomplete-account recovery path still targeted legacy page state, which the current router ignored and redirected back to email entry. The create-user catch also swallowed its error, allowing the confirmation success callback to run afterward.

Test plan

  • npm test -- --run src/common/store/pages/signon/getSignOnRoute.test.ts src/common/store/pages/signon/reducer.test.ts
  • npm run typecheck
  • ESLint on all touched files
  • Prettier check and git diff --check

@changeset-bot

changeset-bot Bot commented Jul 20, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 0605410

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@raymondjacobson raymondjacobson added the preview Deploy a Cloudflare preview for this PR label Jul 20, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🌐 Web preview ready

Preview URL: https://audius-web-preview-pr-14530.audius.workers.dev

Unique preview for this PR (deployed from this branch).
Workflow run

@raymondjacobson
raymondjacobson merged commit f6c6e3b into main Jul 20, 2026
13 checks passed
@raymondjacobson
raymondjacobson deleted the codex/resumable-signup branch July 20, 2026 21:16
raymondjacobson added a commit that referenced this pull request Jul 20, 2026
## Summary

- show a retryable failure state when native account creation fails
- resume Identity-only accounts at handle selection and indexed
incomplete accounts at profile completion
- preserve guest password recovery without advancing when the user edits
their handle
- use the exported common route namespace so the shared web helper
typechecks in mobile

Follow-up to #14530.

## Test plan

- [x] Mobile TypeScript check
- [x] Web TypeScript check
- [x] Mobile focused Jest tests (7 tests)
- [x] Web sign-on route Vitest tests (4 tests)
- [x] ESLint on all touched files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview Deploy a Cloudflare preview for this PR size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant