search

dexlab-io / dexpay-pos

Dexpay: POS, a modularized and extensible point of sale which enables Merchants to accept payments in ETH, DAI and xDAI.
https://www.dexlab.io/
MIT License
27 stars 20 forks source link

Unclear error message on password #134

Closed dimarconicola closed 5 years ago

dimarconicola commented 5 years ago
Screenshot 2019-03-22 at 10 51 33

Should we propose password at least 8 characters long?

dimarconicola commented 5 years ago

Password should be 6 characters long and contain at least one number one uppercase and one lowercase letter.

Alexintosh commented 5 years ago

no

dimarconicola commented 5 years ago

@Alexintosh when trying to create a password these are the messages I get

Screenshot 2019-04-05 at 10 43 29

then

Screenshot 2019-04-05 at 10 43 34

then

Screenshot 2019-04-05 at 10 43 39

I think it would be better to condensate it in one sentence so to inform the user about these constraits as soon as possible in order not to create frustration

perminder-klair commented 5 years ago

@dimarconicola can I get UI for this please

dimarconicola commented 5 years ago

@perminder-klair I think this could be an easy fix "Your password should be at least 6 characters long and contain one number and one uppercase letter"

Screenshot 2019-04-05 at 17 45 04
perminder-klair commented 5 years ago

@dimarconicola sweet sweet design! Adding it

perminder-klair commented 5 years ago

@dimarconicola it's done, what should we do with RED errors? keep them or hide them? Or some other solution you have?