Skip to content

KYC: Fix - Mark the verification attempt as started when the SDK opens - #2336

Open
cmgustavo wants to merge 1 commit into
bitpay:developfrom
cmgustavo:feat/save-status-kyc-attempt-01
Open

KYC: Fix - Mark the verification attempt as started when the SDK opens#2336
cmgustavo wants to merge 1 commit into
bitpay:developfrom
cmgustavo:feat/save-status-kyc-attempt-01

Conversation

@cmgustavo

Copy link
Copy Markdown
Member

A user who abandoned identity verification midway — closing the app while the SumSub SDK still had steps pending — was greeted by the "Get Verified" prompt on relaunch, as if they had never started. SumSub itself resumes correctly: reopening drops the user back exactly where they left off with their documents intact. Only the app's entry point was wrong.

Fix: This calls the new BitPayUser.startKycAttempt RPC right before launching the SDK, which covers both the initial start and every resume.

@cmgustavo cmgustavo changed the title KYC: Feat - Mark the verification attempt as started when the SDK opens KYC: Fix - Mark the verification attempt as started when the SDK opens Aug 25, 2026
@JohnathanWhite
JohnathanWhite requested a balanced review from Copilot August 25, 2026 19:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Marks KYC attempts as started before opening SumSub and adds resume-oriented UI for in-progress verification.

Changes:

  • Adds and invokes the startKycAttempt RPC.
  • Introduces the inProgress UI state across selectors and screens.
  • Extends effect and selector tests.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/api/sumsub/index.ts Adds the attempt-start RPC wrapper.
src/store/sumsub/sumsub.effects.ts Calls the RPC before SDK launch.
src/store/sumsub/sumsub.effects.spec.ts Tests RPC ordering and failure handling.
src/store/sumsub/sumsub.types.ts Adds the inProgress state.
src/store/sumsub/sumsub.selectors.ts Maps and displays in-progress attempts.
src/store/sumsub/sumsub.selectors.spec.ts Tests in-progress state behavior.
src/navigation/tabs/home/components/KycBannerGate.tsx Adds the continuation banner.
src/navigation/bitpay-id/screens/VerifyIdentity.tsx Adds the resume screen and action.
src/navigation/bitpay-id/screens/ProfileSettings.tsx Adds the continuation status pill.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/navigation/tabs/home/components/KycBannerGate.tsx
Comment thread src/api/sumsub/index.ts Outdated
@cmgustavo
cmgustavo force-pushed the feat/save-status-kyc-attempt-01 branch from a9214fc to 36696f1 Compare August 25, 2026 20:43
@cmgustavo
cmgustavo force-pushed the feat/save-status-kyc-attempt-01 branch from 36696f1 to fe94e52 Compare August 26, 2026 14:02
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