Skip to content
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

Account with no password created if double-tap 'Next' button on 'Chose storage' view #11043

Closed
Serhy opened this issue Aug 5, 2020 · 0 comments · Fixed by #11103
Closed

Account with no password created if double-tap 'Next' button on 'Chose storage' view #11043

Serhy opened this issue Aug 5, 2020 · 0 comments · Fixed by #11103

Comments

@Serhy
Copy link
Contributor

Serhy commented Aug 5, 2020

Bug Report

Problem

When creating/recovering with seed phrase account user offered to Choose key storage followed up by creating password.
If to (miss)tap quickly the Next button on Choose key storage then we push empty value for password along with it thus creating accounts with empty (or null) passwords.
No way to log in to such account later.

Note: on iOS there is no Choose storage so far, but bug reproduced there as well.

Note2(optional to reproduce, because it may gone if the first issue fixed): if to navigate back while empty passwords screen is under submission and tap Next again then user winds up on Home screen with Error pop-up No protocol method IMap.-dissoc defined for type string: account not found. So this fatality looks as: from Chose key storage tap Next->Next->Back->Next

Expected behavior

Can not tap Next button twice (or if user does, - it does not bypass Create password screens)

Actual behavior

1123d

(For Next->Next->Back->Next flow)
Screenshot 2020-08-05 at 19 17 33

Reproduction

  • Open Status
  • Generate new key -> Genarate keys -> Next
  • On Choose key storage view tap Next button twice

Additional Information

  • Status version: release 1.5 / develop 1.6
  • Operating System: Android and iOS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant