The delete wallet screens tell key and xprv wallets to rely on a recovery phrase they do not have #529

Open
opened 2026-10-14 04:51:51 +02:00 by clawbot · 0 comments
Collaborator

Where

DeleteWallet and DeleteWalletLostPassword, opened for a key wallet or an xprv wallet. The wording is fixed in src/popup/index.html: line 1127 ("Any funds will be unrecoverable without your recovery phrase.") and lines 1178-1185 ("If you have the recovery phrase for this wallet written down, add the wallet again ...").

What happens

On next (8e52528), release build, Chrome: Settings, [x] on an xprv wallet, then "I have lost my password". Both screens tell the user the wallet's recovery phrase is their way back. Key and xprv wallets have no recovery phrase; their only backup is the private key or extended private key they were imported from. The user is told to look for a phrase that does not exist, and never told that the key is what they need.

What should happen

README, AddWallet: the password hint names "the recovery phrase, the private key or the extended private key, according to the tab". README, DeleteAddress: its route-back text "can name the wallet's own kind of key material". The two delete screens should do the same.

Severity: minor

Definition of done

  • DeleteWallet and DeleteWalletLostPassword name the recovery phrase, the private key or the extended private key, according to the wallet's kind.
  • A test opens both screens for each kind and checks the wording.

Model: opus-5-5

## Where DeleteWallet and DeleteWalletLostPassword, opened for a key wallet or an `xprv` wallet. The wording is fixed in `src/popup/index.html`: line 1127 ("Any funds will be unrecoverable without your recovery phrase.") and lines 1178-1185 ("If you have the recovery phrase for this wallet written down, add the wallet again ..."). ## What happens On `next` (8e52528), release build, Chrome: Settings, `[x]` on an `xprv` wallet, then "I have lost my password". Both screens tell the user the wallet's recovery phrase is their way back. Key and `xprv` wallets have no recovery phrase; their only backup is the private key or extended private key they were imported from. The user is told to look for a phrase that does not exist, and never told that the key is what they need. ## What should happen README, AddWallet: the password hint names "the recovery phrase, the private key or the extended private key, according to the tab". README, DeleteAddress: its route-back text "can name the wallet's own kind of key material". The two delete screens should do the same. Severity: minor ## Definition of done - [ ] DeleteWallet and DeleteWalletLostPassword name the recovery phrase, the private key or the extended private key, according to the wallet's kind. - [ ] A test opens both screens for each kind and checks the wording. Model: opus-5-5
clawbot self-assigned this 2026-10-14 04:51:51 +02:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#529