Skip to content

Conversation

janek26
Copy link
Contributor

@janek26 janek26 commented Oct 16, 2025

BX-2036

Updated the wallet deletion logic to ensure we're using the most up-to-date list of wallets when determining whether to set a new current address. Instead of checking the length of the existing visibleWallets array, we now fetch the latest wallets data and check if there are any remaining wallets before potentially setting a new current address.

What to test

  • On a fresh install, go to settings->wallets, select the only keychain and add a 2nd account. When it asks to set a name, cancel the dialog. Everything should work as expected now, before it was reseting the wallet
  • Delete a wallet and verify that the current address is properly updated to another wallet if available
  • Delete the last wallet and verify the behavior is correct
  • Ensure wallet deletion works properly in all scenarios (single wallet, multiple wallets, etc.)
  • Verify that wallet names and backups are properly deleted

Copy link

linear bot commented Oct 16, 2025

@janek26
Copy link
Contributor Author

janek26 commented Oct 16, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@janek26 janek26 marked this pull request as ready for review October 16, 2025 06:58
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