When the user generates a new wallet there was no check to ensure that the user had actually saved the mnemonic. These changes add a prompt that require the user to input 3 randomly selected elements from the seed phrase to confirm that it is saved.
Implementation
Upon creation of a new wallet (not using the --recover flag) the user is shown their seed phrase and asked to confirm that they have saved it. Once they have they are prompted to input three randomly selected elements from the phrase to confirm that it is saved.
A small bug was fixed which allowed the user to select a wallet from another chain, and another was fixed to add the correct prefix to generated wallets.
Testing
N/A
Notes
I believe the WalletStore changes from the previous PR have resolved the issue with wallets being deleted.
Future work
Allow wallets to be added from WalletConnect or messages to be signed in browser
Motivation
When the user generates a new wallet there was no check to ensure that the user had actually saved the mnemonic. These changes add a prompt that require the user to input 3 randomly selected elements from the seed phrase to confirm that it is saved.
Implementation
Upon creation of a new wallet (not using the
--recoverflag) the user is shown their seed phrase and asked to confirm that they have saved it. Once they have they are prompted to input three randomly selected elements from the phrase to confirm that it is saved.A small bug was fixed which allowed the user to select a wallet from another chain, and another was fixed to add the correct prefix to generated wallets.
Testing
N/A
Notes
I believe the
WalletStorechanges from the previous PR have resolved the issue with wallets being deleted.Future work
Allow wallets to be added from WalletConnect or messages to be signed in browser