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.

Sign up for GitHub

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

Jump to bottom

Change password #26

Open

braidsencurls wants to merge 1 commit into anantjain6:master from braidsencurls:change-password

braidsencurls commented Oct 3, 2020

No description provided.


          Change password

f9f24ea

braidsencurls mentioned this pull request

Add option to change password #23

Open

Owner

anantjain6 commented Oct 5, 2020

@braidsencurls Thanks for your contribution

I have the following comments:-

"Password" can not be model as per MVC architecture
In "change-password.jsp" for new password, you have instructed that it should contain an upper case and a lower case, but as per your code, it should contain a digit/numeric also.
On entering the wrong old password you are showing error message "Error changing Password" which don't describe the user about the mistake he/she has done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet