Skip to content

Improve documentation for cases when migrating from FGAP:V1 to V2#48702

Open
vramik wants to merge 1 commit intokeycloak:mainfrom
vramik:48588-improve-docs-about-fgap
Open

Improve documentation for cases when migrating from FGAP:V1 to V2#48702
vramik wants to merge 1 commit intokeycloak:mainfrom
vramik:48588-improve-docs-about-fgap

Conversation

@vramik
Copy link
Copy Markdown
Contributor

@vramik vramik commented May 5, 2026

Closes #48588

@vramik vramik force-pushed the 48588-improve-docs-about-fgap branch 2 times, most recently from 199aa1f to 67f4034 Compare May 5, 2026 09:25
@vramik vramik marked this pull request as ready for review May 5, 2026 09:27
@vramik vramik requested a review from a team as a code owner May 5, 2026 09:27
pedroigor
pedroigor previously approved these changes May 5, 2026
@pedroigor pedroigor enabled auto-merge (rebase) May 5, 2026 14:09
auto-merge was automatically disabled May 6, 2026 09:52

Head branch was pushed to by a user without write access

@vramik vramik force-pushed the 48588-improve-docs-about-fgap branch from 67f4034 to 39cbbed Compare May 6, 2026 09:52
@vramik
Copy link
Copy Markdown
Contributor Author

vramik commented May 6, 2026

@pedroigor @sguilhen the new validation for decisionStrategy revealed a bug in PolicyTypeService (where fields param was silently ignored) and ModelToRepresentation (where only resourcesData and scopesData were populated, but resources and scopes were left null). PTAL

If preferred, I can extract the fix into separate PR and make this one only about docs improvement, wdyt?

@pedroigor
Copy link
Copy Markdown
Contributor

@vramik Makes sense to keep here only the doc updates.

@vramik
Copy link
Copy Markdown
Contributor Author

vramik commented May 6, 2026

@vramik Makes sense to keep here only the doc updates.

#48749

@vramik vramik force-pushed the 48588-improve-docs-about-fgap branch from 39cbbed to 3839027 Compare May 6, 2026 12:30
@vramik vramik requested a review from pedroigor May 6, 2026 12:30
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.

Fine Grained Authorization v2 : does not work PUT https://i<keycloak>/admin/realms/<realm>/admin-permissions/groups/<group-id>

3 participants