-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Issue #44: Redesign forgot/reset/confirm pages #83
Issue #44: Redesign forgot/reset/confirm pages #83
Conversation
Update from original
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested your code on my local and it looks great.
However, the reset password form doesn't check if the new password contains a uppercase character because the regex does not check uppercase.
I have fixed that in another pull request.
@felixApplatix Good you found it. I see it is general issue and we just validate if password contains any of uppercase or lowercase letters. Our requirement is to have uppercase AND lowercase letters. |
…ppercase AND lowercase letters.
* argoproj#44 Redesign sign-up page * argoproj#44 Redesign forgot password page * argoproj#44 Redesign confirm forgot password page * argoproj#44 Redesign confirm reset password page * argoproj#44 Redesign reset password page. Adjust buttons and cleanup. * argoproj#44 Redesign confirm setup page * argoproj#44 Adjust boxes to small screens * argoproj#44 Update custom regexp validator to allow only texts with uppercase AND lowercase letters.
* argoproj#44 Redesign sign-up page * argoproj#44 Redesign forgot password page * argoproj#44 Redesign confirm forgot password page * argoproj#44 Redesign confirm reset password page * argoproj#44 Redesign reset password page. Adjust buttons and cleanup. * argoproj#44 Redesign confirm setup page * argoproj#44 Adjust boxes to small screens * argoproj#44 Update custom regexp validator to allow only texts with uppercase AND lowercase letters.
* argoproj#44 Redesign sign-up page * argoproj#44 Redesign forgot password page * argoproj#44 Redesign confirm forgot password page * argoproj#44 Redesign confirm reset password page * argoproj#44 Redesign reset password page. Adjust buttons and cleanup. * argoproj#44 Redesign confirm setup page * argoproj#44 Adjust boxes to small screens * argoproj#44 Update custom regexp validator to allow only texts with uppercase AND lowercase letters.
Fixed unit tests. Testing Done: 1. Unit tests pass 2. Added an example sensor. It failed prior to this change. Succeeded after. Closes argoproj#82
Fixed #44