8000 Reload page when selecting a realm to force updating user permissions by pedroigor · Pull Request #33500 · keycloak/keycloak · GitHub
[go: up one dir, main page]
Skip to content

Reload page when selecting a realm to force updating user permissions - #33500

Closed
pedroigor wants to merge 1 commit into
keycloak:mainfrom
pedroigor:issue-31638
Closed

Reload page when selecting a realm to force updating user permissions#33500
pedroigor wants to merge 1 commit into
keycloak:mainfrom
pedroigor:issue-31638

Conversation

@pedroigor
@pedroigor pedroigor commented Oct 2, 2024 •
edited
Loading
Copy link
Copy Markdown
Contributor

Closes #31638

  • Not yet fixing the error page when login in to the master realm (or any other realm the user hasn't access). We could think about a better page/message asking the user to select another realm
  • When selecting the realm, the page is reloaded to make sure permissions are properly set and the user can access the other realms he has access to

8000
@pedroigor
Reload page when selecting a realm to foce updating user permissions
6d6d3b8 
Closes keycloak#31638

Signed-off-by: Pedro Igor <pigor.craveiro@gmail.com>
@pedroigor
pedroigor requested a review from a team as a code owner October 2, 2024 23:56
@pedroigor pedroigor mentioned this pull request Oct 2, 2024
Closed
2 tasks done
sguilhen
sguilhen approved these changes Oct 3, 2024
View reviewed changes
@sguilhen sguilhen left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This gets a +1 from me. Tested locally and was able to switch to the realms the user has permissions to operate and manage them.

@pedroigor pedroigor changed the title Reload page when selecting a realm to foce updating user permissions Reload page when selecting a realm to force updating user permissions Oct 3, 2024
@pedroigor
pedroigor commented Oct 4, 2024
Copy link
Copy Markdown
Contributor Author

@sguilhen @HannahBrown Closing in favor of #32974.

@pedroigor pedroigor closed this Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@sguilhen sguilhen sguilhen approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Error when non-admin user accesses admin console

2 participants

@pedroigor @sguilhen
0