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

Let managers reset user's password #2500

Closed
bertocq opened this issue Mar 1, 2018 · 7 comments · Fixed by AyuntamientoMadrid/consul#1370
Closed

Let managers reset user's password #2500

bertocq opened this issue Mar 1, 2018 · 7 comments · Fixed by AyuntamientoMadrid/consul#1370
Assignees
Labels
Coadministration Management, Moderation and Valuation

Comments

@bertocq
Copy link
Collaborator

bertocq commented Mar 1, 2018

What we need

The manager should have a button to "reset password" for user:

1_reset_password

There are two options:

  1. If the user has an email present (and its one that he controls)
    Show a form with the button "Send reset password email" So the user will get a link that he can use to stablish a new password. This is useful when the user didn't have an email or not one that he controls... and the manager just fixed it for him (this is already described at issue Let managers update a user's email #2499)

2_reset_password

  1. If the user doesn't have an email (or doesn't control/use it)
    This would show the Manager a form with:
  • A "Password" input that shows in plain text the password and lets him type a new one by request of the user or his own will. There should be a "help text" next to it explaining that after saving it we'll offer him to "Print password" so there's no need to write it down right now.
  • A "Generate random password" next to the input would be great, in the case the citizen doesn't need to have a "memorable" or "human readable" password.
  • A "Save button"

3_reset_password

After hitting "Save" the Manager should have a "Print password" button that allows the manager to easily print the new password (we're not storing it in plain text... so somehow "store" it in session or somewhere) as an empty html page or pdf or... whatever its easier!

4_reset_password

@bertocq bertocq added Coadministration Management, Moderation and Valuation ⭐️ labels Mar 1, 2018
@bertocq bertocq added this to Next in Roadmap Mar 1, 2018
@bertocq bertocq changed the title Let managers reset user's password via email Let managers reset user's password Mar 1, 2018
@iagirre
Copy link

iagirre commented Mar 12, 2018

I'll do it

@MariaCheca MariaCheca moved this from Next to Doing in Roadmap Mar 13, 2018
@iagirre
Copy link

iagirre commented Mar 14, 2018

@decabeza I have two doubts regarding the UI:

  1. Where should I put the buttons for reseting the password and showing the password form? Should I follow the same style as described in Let managers update a user's email #2499 (both in the rigth side of the square)?
  2. If the buttons are placed in the grey square, they must appear every time or only when the user is in management/account?

@decabeza
Copy link
Collaborator

Hi @iagirre! I updated the issue and included some mockups 😉

@bertocq
Copy link
Collaborator Author

bertocq commented Mar 22, 2018

Another "automagically closed" that shouldn't :(

Please @iagirre a backport to consul whenever you have the time to entirely close this issue 😃

@bertocq bertocq reopened this Mar 22, 2018
@MariaCheca MariaCheca moved this from Doing to Review in Roadmap Apr 2, 2018
@voodoorai2000
Copy link
Member

Closed via #2548

@voodoorai2000 voodoorai2000 moved this from Review to Done in Roadmap Apr 5, 2018
@KrzysztofMadejski
Copy link

Where was this implemented? Can't find this feature..

@decabeza
Copy link
Collaborator

Hi @KrzysztofMadejski this feature was implemented on #2548 (you can see some attached gifs of how it works).

The URL to access to Management section is /management you can access with an admin user or a user with manager role.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Coadministration Management, Moderation and Valuation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants