-
Notifications
You must be signed in to change notification settings - Fork 490
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
API: add endpoint for listing users #3009
Comments
Having this endpoint makes documenting the conversion from Shib to builtin for #2915 easier.
@pdurbin - is the "@" coming from the export, or something that's part of the identifiers in your test set? aka - "@dataverseAdmin" vs "dataverseAdmin"? |
@pameyer it's sort of a long story. See #1151 #1445 #1705 and probably other issues. The answer to your specific question is that Otherwise, I hope the data returned looks ok and useful. I did go ahead and push 0b212f8 which is the code I was playing with to dump the JSON above. |
Passing to QA. The new API endpoint is documented as |
OK, works, closing. |
In chat yesterday we discussed adding an API for listing users. Technically, I already stubbed something out in 7ea6a01 but in a standup meeting this morning I advocated to make it more real, especially so I can explain how to list users before attempting to convert one from being a Shibboleth user to being a local/builtin user (#2915). I'm playing around with what the JSON would look like and I'm thinking of something along these lines:
The text was updated successfully, but these errors were encountered: