-
Notifications
You must be signed in to change notification settings - Fork 51
Fix change password functionality #11
Comments
i think this can be a good code in task |
added this as a task |
using abov link (basically a search you can find where to make this change ) ..its basically in 2 files .. when you setup the password or change the password |
Hi, I am more inclined to have a text change listner on each text boxes and disable the button until both are equal to each other. Or do you want to validate once the button is clicked? Which would be better? Regards P.S. I am working on this task through google code in |
@ishanqut where is the Pull request ? |
I approved the task .. can you double check ? On Fri, Dec 18, 2015 at 4:11 PM, ahmedsabie notifications@github.com
|
@Neetuj I didn't claim the task yet because my current task is still being reviewed. Can I claim it after? |
Hi sorry I am not sure which task you are talking about then this was already claimed by Ishan sorry for the confusion .. but if you are working on another task then this
On Fri, Dec 18, 2015 at 4:33 PM, ahmedsabie notifications@github.com
|
oh yeah I am referring to this task, but I still see it on the Google Code in website. I didn't claim it because I am doing another task which is being reviewed, so I thought I would make the pull request first. What should I do? |
This is already done by ishan and approved Sent from my iPhone
|
ah ok I didn't know that. Thanks for letting me know. |
Fix the change password functionality so that it checks that the New Password field and Confirm Password field match. Currently, the form is not checking whether the input to these 2 fields match.
The text was updated successfully, but these errors were encountered: