Skip to content

feat(ramp): route Money Account onboarding stages - #35894

Draft
roz0n wants to merge 32 commits into
mainfrom
feat/TRAM-3923-neobank-routing
Draft

roz0n wants to merge 32 commits into
mainfrom
feat/TRAM-3923-neobank-routing

Conversation

@roz0n

@roz0n roz0n commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Description

Routes Money Account onboarding from the stage returned by RampsController.hydrateNeobankStore.

The router only selects a screen. It does not launch SumSub. Missing terms go to Get Pix Key. KYC stages go to a dedicated KYC page whose focus hook initializes or resumes Iron KYC. Accepted terms are also passed to that page so the terms screen does not present the SDK directly.

The flow refreshes on app resume and fails closed for unknown or failed lookups.

State mapping

Core stage Mobile destination
NoUser Get Pix Key terms
VendorTermsRequired Get Pix Key terms
ProviderTermsRequired Get Pix Key terms
EmailOtpRequired Email and OTP surface
KycNotStarted Dedicated KYC page
KycStartedIncomplete Dedicated KYC page
KycNeedsReview Dedicated KYC page
KycPending Processing surface
WalletNotSigned Processing surface
AutorampNotCreated Processing surface
AutorampPending Processing surface
AutorampCreated Money home
KycRejected Error surface
LookupFailed or unknown Retryable error surface

Dependencies

Tests

  • 5 focused suites passed
  • 36 tests passed
  • Mobile TypeScript check passed
  • ESLint passed for changed files

Related issue

roz0n and others added 6 commits September 1, 2026 11:46
Replace the client-side KYC API fetch in useKycDisclaimers with
KycController.loadDisclaimers so Iron/MoonPay vendor terms use a single
Engine source of truth. Wires preview @metamask/kyc-controller with
minimal KycService/KycController init for disclaimer loading only.

TRAM-3978

Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Sébastien Van Eyck <sebastien.vaneyck@consensys.net>
@roz0n roz0n self-assigned this Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@github-actions github-actions Bot added size-XL pr-not-ready-for-e2e Skip E2E and block merging. Remove this label once the PR is ready to run the E2E tests. labels Sep 8, 2026
@roz0n roz0n added the blocked label Sep 8, 2026
@metamask-ci metamask-ci Bot added the team-money-movement issues related to Money Movement features label Sep 8, 2026
@socket-security

socket-security Bot commented Sep 8, 2026

Copy link
Copy Markdown

No dependency changes detected. Learn more about Socket for GitHub.

👍 No dependency changes detected in pull request

@roz0n
roz0n force-pushed the feat/TRAM-3923-neobank-routing branch from 87f370e to 008e628 Compare September 8, 2026 15:19
@roz0n
roz0n changed the base branch from main to feat/TRAM-3922-kyc-engine-wiring September 8, 2026 15:19
@github-actions github-actions Bot added size-L and removed size-XL labels Sep 8, 2026
jiexi and others added 11 commits September 8, 2026 11:58
…ntroller' into feat/vba-kyc-disclaimers-from-controller
Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Sébastien Van Eyck <sebastien.vaneyck@consensys.net>
Signed-off-by: Sébastien Van Eyck <sebastien.vaneyck@consensys.net>
Signed-off-by: Sébastien Van Eyck <sebastien.vaneyck@consensys.net>
Signed-off-by: Sébastien Van Eyck <sebastien.vaneyck@consensys.net>
@roz0n
roz0n force-pushed the feat/TRAM-3922-kyc-engine-wiring branch from bb5c4c7 to c787d07 Compare September 10, 2026 14:11
@roz0n
roz0n force-pushed the feat/TRAM-3923-neobank-routing branch from 008e628 to b0a98a6 Compare September 10, 2026 14:11
roz0n and others added 2 commits September 11, 2026 10:30
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@roz0n
roz0n force-pushed the feat/TRAM-3923-neobank-routing branch from b0a98a6 to 26538df Compare September 11, 2026 14:53
@sonarqubecloud

Copy link
Copy Markdown

@roz0n
roz0n removed this pull request from stack #35900 September 16, 2026 13:45
Base automatically changed from feat/TRAM-3922-kyc-engine-wiring to main September 17, 2026 07:48
@metamask-ci metamask-ci Bot added the INVALID-PR-TEMPLATE PR's body doesn't match template label Sep 17, 2026
@metamask-ci

metamask-ci Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

PR template — items to address before "Ready for review"

Warnings — informational, address before merging:

See docs/readme/ready-for-review.md for the full Definition of Ready for Review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked INVALID-PR-TEMPLATE PR's body doesn't match template pr-not-ready-for-e2e Skip E2E and block merging. Remove this label once the PR is ready to run the E2E tests. size-L team-money-movement issues related to Money Movement features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants