Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Missing password validation during onboarding #415

Closed
odb366 opened this issue Jun 12, 2018 · 7 comments
Closed

Missing password validation during onboarding #415

odb366 opened this issue Jun 12, 2018 · 7 comments

Comments

@odb366
Copy link
Contributor

odb366 commented Jun 12, 2018

During onboarding process users are asked to create a pass for unlocking ZAP. The pass should be 8 or more characters, but users are able to create one with less than 8 characters and proceed. Inline validation is missing. Consequently, on the last step of the onboarding they cannot proceed, and are unable to start ZAP.

Update: the user can even proceed without entering the confirmation password on this screen.

screenshot 32

@mrfelton
Copy link
Member

This appears to still be an issue after merging of #424.

User can proceed past this page without setting a password at all, which prevents the user from being able to complete the wallet setup process as LND requires a password

@odb366
Copy link
Contributor Author

odb366 commented Jun 28, 2018

Wow, how did I missed that. Thanks @mrfelton. Fix incoming.

@odb366
Copy link
Contributor Author

odb366 commented Jun 28, 2018

Another usability issue is that we don't have a back button on this screen. Should I add back functionality to the autopilot step?

@JimmyMow
Copy link
Member

No. That's more complex than you think. We'd have to shut down our LND process we just started. No back button there

@odb366
Copy link
Contributor Author

odb366 commented Jun 28, 2018

@JimmyMow Ok

@odb366
Copy link
Contributor Author

odb366 commented Jun 28, 2018

Issue fixed, closing

@odb366 odb366 closed this as completed Jun 28, 2018
@odb366
Copy link
Contributor Author

odb366 commented Jun 28, 2018

The issue is reopened because it is possible to proceed with 2 different passwords ..

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants