Skip to content

Change password #2959

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

Merged
merged 6 commits into from
Feb 19, 2018
Merged

Change password #2959

merged 6 commits into from
Feb 19, 2018

Conversation

di
Copy link
Member

@di di commented Feb 16, 2018

Fixes #999, fixes #2831.

@nlhkabu I don't think this needs much (if any) styling, could you take a quick look?

screen shot 2018-02-16 at 11 29 33 am

@nlhkabu
Copy link
Contributor

nlhkabu commented Feb 16, 2018

Yep, will do @di. Thanks! :)

@nlhkabu
Copy link
Contributor

nlhkabu commented Feb 16, 2018

hi @di - I'm getting a template error - but your new code looks ok to me... any ideas?

screenshot from 2018-02-16 19-56-09

@di
Copy link
Member Author

di commented Feb 16, 2018

@nlhkabu I had a typo that I force-pushed and fixed, you'll need to pull down this branch again. Sorry!

@di di force-pushed the change-password branch from 64ff234 to 812ec84 Compare February 16, 2018 20:13
@nlhkabu
Copy link
Contributor

nlhkabu commented Feb 16, 2018

no problem, thanks for the fix!

@di di requested a review from brainwane February 16, 2018 22:01
@nlhkabu
Copy link
Contributor

nlhkabu commented Feb 17, 2018

Updated @di:

screenshot from 2018-02-17 07-10-03

@di di merged commit e310460 into master Feb 19, 2018
@di di deleted the change-password branch February 19, 2018 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Manage Account: Change Password Send confirmation email when password changed
2 participants