[5.5] Keystone: fix 2 bugs

[soralit]

Description

This PR fixed 2 bugs with Keystone:

1. keystone accounts will disappear when reset password.
2. keystone accounts page will increase 1 when scanning wrong wallet sync qrcode.

Checklist

• There is a related GitHub issue
• Tests are included if applicable
• Any added code is fully documented

Screenshots/Recordings

If applicable, add screenshots or recordings to visualize the changes

Issue

Progresses https://github.com/MetaMask/qr-hardware-wallets/issues/7
Progresses https://github.com/MetaMask/qr-hardware-wallets/issues/6

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[gantunesr]

Thanks @soralit! I'm leaving some questions for the moment

[soralit]

Hi @gantunesr , PR was updated. I found the restoring process was duplicated in two place so I combined them into one, what do you think?

[cortisiko]

@soralit I am unable to log in after a password change. Basically, when I attempt to change my password with biometrics log-in disabled I get an error “invalid password”.

To reproduce:

import your wallet
go to the privacy and security page
tap on the “change password” button
enter your current password to proceed
now you should be taken to the create password screen
enter a new password
before creating the new password toggle off “log in with biometrics”
proceed and create that new password
After the password change was successful
kill the app
relaunch the app
login with your new password. Notice the error

I reproduced without binding my keystone device: http://recordit.co/jt2b4IjCxm

And after I bind my keystone device: http://recordit.co/iKkwuYgxpP

[AlexJupiter]

Hey @soralit any update on this at all? :)

[soralit]

Hi @AlexJupiter @cortisiko , really sorry for the late reply. I just fixed this issue. Thanks for your feedback.

[gantunesr]

LGTM

[cortisiko]

This PR is 🌮 🌮