-
Notifications
You must be signed in to change notification settings - Fork 196
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
graph: add identities property to non-admin response of GetUsers #9819
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
Hm, I just noticed that the demo users don't have the 'Identities' value configured yet. I'll add a fix. |
|
Currently only federated users are guaranteed to have the I guess we should adjust that so that the |
To make sure they get and 'identities' property in the graph responses.
|
tested it. works fine 👍 I'll merge your PR and change my tests #9812 |
graph: add identities property to non-admin response of GetUsers
graph: add identities property to non-admin response of GetUsers
graph: add identities property to non-admin response of GetUsers
Fixes: #9813