Skip to content
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

T695 reset user pass #192

Merged
merged 9 commits into from Nov 11, 2019
Merged

T695 reset user pass #192

merged 9 commits into from Nov 11, 2019

Conversation

ghost
Copy link

@ghost ghost commented Oct 3, 2019

this adds a new password reset feature the the admin user view page, passwords are randomly generated


ref T695

  • I have signed the CLA

this adds a new button when viewing a user as an admin, that will
generate and store a new password for the user
@thebaer thebaer added this to the 0.11 milestone Oct 10, 2019
Move logic into template, rather than add another field to the page.

Ref T695
This resets user password to something random that also reminds the user
they should change it immediately after logging in, instead of a
completely random jumble of characters.

Ref T695
This also adjusts the style and includes the user's password, so the
admin can easily notify them.

Ref T695
Also updates some whitespace in the JS.

Ref T695
Previously, this had bad template logic and showed the wrong email address.

Ref T695
@thebaer
Copy link
Member

thebaer commented Nov 11, 2019

Tweaked some copy and design, and made it so the admin can see user's email address (if they have one) in order to get in touch. This also uses passgen.NewWordish(), instead, for password generation.

@thebaer thebaer merged commit 5839c2a into develop Nov 11, 2019
@thebaer thebaer deleted the T695-reset-user-pass branch November 11, 2019 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant