Skip to content

Conversation

@jvsena42
Copy link
Member

@jvsena42 jvsena42 commented Jan 29, 2026

Fixes #669

This PR sets backupVerified to true when a wallet is successfully restored, preventing the unnecessary backup warning sheet from appearing after restore.

Description

When a user restores a wallet using their recovery phrase, they clearly have access to their backup. The app was incorrectly showing the "Wallet Backup" bottom sheet after restore because backupVerified remained false. This PR adds an onSuccess handler to the restoreWallet function that marks the backup as verified.

Preview

Screen_recording_20260129_072648.webm

QA Notes

1. Restore wallet flow

  1. Wipe the app data or use a fresh install
  2. Tap "Restore wallet"
  3. Enter a valid 12-word recovery phrase
  4. Complete the restore process
  5. Verify the "Wallet Backup" bottom sheet does NOT appear

2. New wallet flow (regression)

  1. Wipe the app data or use a fresh install
  2. Create a new wallet
  3. Verify the "Wallet Backup" suggestion still appears (expected behavior for new wallets)

@jvsena42 jvsena42 self-assigned this Jan 29, 2026
@claude

This comment has been minimized.

@jvsena42 jvsena42 marked this pull request as draft January 29, 2026 10:11
@jvsena42 jvsena42 marked this pull request as ready for review January 29, 2026 10:31
@jvsena42 jvsena42 requested a review from ovitrif January 29, 2026 10:32
@claude
Copy link

claude bot commented Jan 29, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

Copy link
Collaborator

@ovitrif ovitrif left a comment

Choose a reason for hiding this comment

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

lgtm with one nit

@ovitrif ovitrif merged commit 149a991 into master Jan 30, 2026
15 checks passed
@ovitrif ovitrif deleted the fix/disable-timed-sheet-after-restore branch January 30, 2026 04:03
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.

Wallet Backup sheet shown after restoring wallet

3 participants