-
Notifications
You must be signed in to change notification settings - Fork 376
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
UI: Allow org admin to hide change email and change password #4660
Comments
We need to determine if the fact that LDAP auth is used is surfaced anywhere for the FE. Edit: yes - the UI can use the |
Also, this would likely be helpful in scenarios where a user authentication is OAuth. |
This might need some further thought. I'm not entirely certain that preventing OAuth users from changing their email address is correct. Let's take a scenario here:
In this case:
So maybe, what we should do is to re-add the ability to change your email as an OAuth user, but we should clear the ID from the OAuth provider when an email changes? |
As long as we're not doing it silently, I'm in favour of this option. |
The change here did not fully resolve this issue since changing one's email requires the user's password. OAuth users will likely not have a password set so we need to find another workflow for this. |
Organizations using LDAP don't need the "Change Email" and "Change Password" UI elements.
The text was updated successfully, but these errors were encountered: