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 update a user's email #2499

Open
bertocq opened this issue Mar 1, 2018 · 7 comments
Open

Let managers update a user's email #2499

bertocq opened this issue Mar 1, 2018 · 7 comments
Labels
Coadministration Management, Moderation and Valuation

Comments

@bertocq
Copy link
Collaborator

bertocq commented Mar 1, 2018

What we need

At /management/account the manager should be able to update the email to a user account.

  • Shows user's email or "No-email user".
  • For "change user" red color link you can use delete CSS class.
  • In the edit form, the document type and number fields can be disabled.

Notes

Beware some users might not even have an email.

Screenshots

Info shows "email" and "edit user" button

management_info

New form to change email

management_edit

@bertocq bertocq added the Coadministration Management, Moderation and Valuation label Mar 1, 2018
@bertocq bertocq changed the title Let managers modify an user's email Let managers update an user's email Mar 1, 2018
@bertocq bertocq added the ⭐️ label Mar 1, 2018
@bertocq bertocq added this to Next in Roadmap Mar 1, 2018
@voodoorai2000 voodoorai2000 changed the title Let managers update an user's email Let managers update a user's email Mar 9, 2018
@raul-fuentes
Copy link
Contributor

I'll do this one :)

@MariaCheca MariaCheca moved this from Next to Doing in Roadmap Mar 12, 2018
@raul-fuentes
Copy link
Contributor

@decabeza I need some advise. Where do you think that its better to locate the form in /management/account?

As an in line form in the grey rectangle, switching the email into a text_field? ,
or as an standard form underneath the grey rectangle? (and if so, where sould be the button for delete account be located?)

Thanks in advance

@decabeza
Copy link
Collaborator

Hi @raul-fuentes I updated the issue with more info and screenshots 😉

@raul-fuentes
Copy link
Contributor

Hello @bertocq!

I have a question, when a user's e-mail is changed, a confirmation email is sent and until confirmation is made, the user's email is not modified, I keep this behavior or I simply change the email without sending the email mail confirmation?

Thanks in advance

@bertocq
Copy link
Collaborator Author

bertocq commented Mar 14, 2018

Thanks for the question @raul-fuentes, when in doubt it's always best to ask before taking a path in the code. 👍 👏

I've asked product and the best is to keep the existing behaviour. That means sending the confirmation email message to the new email address and waiting for the user to click in the message link to make the new email address change effective.

@raul-fuentes
Copy link
Contributor

Thank you!

@javierm
Copy link
Member

javierm commented May 24, 2020

Reopening since it was requested in #3546 as well.

@javierm javierm reopened this May 24, 2020
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

No branches or pull requests

5 participants