Blockstream / green_qt

Blockstream Green App
GNU General Public License v3.0
195 stars 32 forks source link

Confusing behavior when PIN is entered incorrectly #8

Closed MarioGibney closed 4 years ago

MarioGibney commented 4 years ago

Ubuntu 18.04.3 LTS Green 0.0.1

After entering one's PIN incorrectly, there is not option to log in to that wallet using mnemonic. The app will only display this page when that wallet is selected:

Screenshot from 2020-02-21 17-54-53

It’s not frozen or anything, but there’s not option to enter my mnemonic from here instead. Seems a bit confusing and unclear.

Then when i go to restore it, it is listed as a new wallet on the app instead of the same one. In this screenshot, all three bitcoin wallets are actually the same wallet:

Screenshot from 2020-02-21 17-57-32

domegabri commented 4 years ago

so you expect to have an option to restore once pin attempts are 0 (reasonable) but also don't restore the same wallet multiple times? at the moment you can restore the same wallet N times even without burning your pin attempts.

Implementing a restore policy means we leave out some edge cases (e.g. user forgets his pin, doesn't know he needs 3 wrong attempts to delete it and there's no delete wallet feature. App policy doesn't allow him to restore because there's already that wallet restored).

MarioGibney commented 4 years ago

I would expect one of these two things to happen:

The second options seems cleaner, IMO.

The main thing that definitely needs fixing is that right now, any time you enter a PIN incorrectly three times, then you have a wallet listed there that you can never access (because entering that same wallet with a mnemonic will appear as a new wallet in that list).

domegabri commented 4 years ago

yes definitely needs to drop from the list wallets with no attempts left. And "no attempts left" view can lead to restore. Appreciate the feedback

alessandro-saglimbeni commented 4 years ago

fixed in 2234b4fa1a08f6a450268611f1101c861f3dfd30 closing