-
Notifications
You must be signed in to change notification settings - Fork 881
capture "full name" from the SSO identity provider + "new user" form #11704
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
Comments
This would be great to get the "DisplayName" from something like MS Entra Id user profiles to populate as this is later on used for the commit messages. From a user experience point of view, onboarding with no additional steps is ideal. |
@bpmct I would not mind working on this feature? Or is this on the roadmap for completion. I would also note that User Profiles should be readonly for OIDC users. |
Hey @michaelbrewer. We'll be working on this as part of our "manage coder via IaC + identify providers" effort this quarter |
Slotting this into our sprint starting on Monday as a high priority. When we implement fields like this down the road, we'll be sure it can be passed in from the identity provider as this is how all of our enterprise customers will want it to work |
This would be awesome. And potentially allow more fields to be mapped into coder like the profile photo, display name, city etc. This helps various use case:
Not all providers have all of these or requires mappings. |
@bpmct should we break out the second piece
into a separate ticket? |
Closing as completed. #13490 can be used to track the new user form |
Awesome will test out "CODER_OIDC_NAME_FIELD" mapping when it is available. |
We recently added support for a "full name" in Coder, which is in the account settings and passed into Terraform. #11639 #11583. Could we also add support for the identify provider (OIDC or GitHub) from a configurable scope typically name?
Could we also add support for optionally specifying the full name in the "new user" form?
The text was updated successfully, but these errors were encountered: