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

#29266 Improve documentation on ClientRolemappingsRessource #29344

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sprehn-ero
Copy link

This will apply the requested changes in admin rest api documentation.
ClientRole Mappings can be applied to both users and groups.
More specific naming and description on Parameter client-id.

POST /admin/realms/{realm}/groups/{group-id}/role-mappings/clients/{client}
I did not find a fix to specify the body argument RoleRepresentation -> List[RoleRepresentation].
The type of this parameter is List, so the description should be automatically applied by the framework.

@sguilhen
Copy link
Contributor

sguilhen commented May 9, 2024

Thanks @sprehn-ero , it does look good to me. Can you please sign-off your commit (see https://github.com/keycloak/keycloak/blob/main/CONTRIBUTING.md#developers-certificate-of-origin) and also edit the commit message with a reference to the original issue (Closes #29266)

Closes keycloak#29266

Signed-off-by: Sebastian Prehn <sebastian.prehn@ero.eu>
@sprehn-ero sprehn-ero force-pushed the bugfix/29266-admin-rest-api-documentation branch from dbd2fa4 to 53b25ce Compare May 13, 2024 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants