McGill-CSB / PHYLO

a gaming framework to align genomic data
phylo.cs.mcgill.ca/edge
Other
11 stars 14 forks source link

OpenPhylo user registration clears all fields when password conformance checks fail #121

Closed movermeyer closed 6 years ago

movermeyer commented 6 years ago

When you register for a new user in the OpenPhylo UI, there is no indication what is required for the password (ex. >=8 characters). It only checks the conformance server side: reg1

Annoyingly, when you password doesn't conform to the password rules, it clears all of the fields in the form (not just the password ones), and gives you the message in an alert box.

reg2

Ideally, conformance to the password policies would be checked on the client side, and the error message would be displayed in-line with the password field.

The part that most annoyed me is having to enter everything again, instead of just correcting the password field. Admittedly this is a minor annoyance. But it causes unnecessary negative emotions just as a user is beginning to use Phylo.

Animated Gif version

akashzcoder commented 6 years ago

Confirmation is already on the client side, but yes form details don't have to be reset. We will resolve that today. This is the first of the many sprint releases that we will be doing for OpenPhylo before the final release.

movermeyer commented 6 years ago

Thanks. I've updated the title.

akashzcoder commented 6 years ago

I have updated this along with better and more specific validation messages. Thank you very much! :)