vegaprotocol / vegawallet-browser

https://vegaprotocol.github.io/vegawallet-browser/
MIT License
3 stars 0 forks source link

Allow user to delete / remove a wallet #501

Closed dexturr closed 6 months ago

dexturr commented 12 months ago

Story

As a user I want to be able to delete my wallet **So that I can import a new one

Linked tasks

Acceptance Criteria

Tasks

Sketch

Notes

barnabee commented 11 months ago

Delete seems like the wrong word, perhaps "remove" instead?

dexturr commented 11 months ago

@barnabee Remove feels a bit too soft to me, if you're removing a mnemonic we need it to be obvious that the user needs to have stored thier mnemonic to add the wallet again in future (until multi wallet). Perhaps solvable with a dialog explaining all of this.

barnabee commented 11 months ago

@dexturr I'd rather be "too soft" than wrong, given that a wallet cannot be deleted.

I think as you suggest the answer here is that the confirmation dialog when removing a wallet states that once it is removed the wallet cannot be recovered/re-added without the seed phrase.

Relatedly, we should also make sure to prioritise these issues that enable a use to see their secret seed phrase and private keys: