-
Notifications
You must be signed in to change notification settings - Fork 66
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Clear saved accounts #34
Comments
Thanks for the suggestion. We will need to implement a log out flow for an individual account whereby a user verifies they know their seed phrase before clearing their accounts. |
Ok so rather than have 1 "clear all" button, have individual buttons that show a "do you know or have safely stored your seedphrase for this account?" confirmation popup before clearing. |
Yes indeed |
We should ask them to either type in their seed phrase or type in "skip" |
There could be an option to sign out of all accounts that would require them to type something longer like "i understand log me out" |
Yea maybe even just secondary confirmation and just use clear() Any thoughts on encrypting all data stored in local storage for additional security? |
@maebeam would it be ok just to have a single button to remove all, using the same @carsenk probably best to create separate issue for I have a feeling it may be a conscious decision why they haven't encrypted all data so would be good to get that background info. |
Pubkey data maybe I was thinking, but yea really kind of just pointless, nevermind 👍 |
@maebeam please check copy & flow below ones happy with that side I will finalise and submit at PR copy in confirmation screen:
|
Looks close enough! Open up a PR and I can push a tweak to your branch if need be. Thanks for doing this tijn! |
pr #39 |
Users should be able to clear the logins saved in Identity & wipe localStorage.
Especially when using on a shared computer.
I added a PoC - let me know if you would like me to PR this.
The text was updated successfully, but these errors were encountered: