8000 make group select required when specific group is selected by edewit · Pull Request #38768 · keycloak/keycloak · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

make group select required when specific group is selected #38768

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

Merged
merged 3 commits into from
Apr 11, 2025

Conversation

edewit
Copy link
Contributor
@edewit edewit commented Apr 9, 2025
  • make group select required when specific group is selected
  • small refactor to make labels not dependant on route

fixes: #38767

edewit added 2 commits April 9, 2025 07:38
fixes: keycloak#38767
Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>
Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>
@edewit edewit requested a review from a team as a code owner April 9, 2025 05:40
@edewit edewit self-assigned this Apr 9, 2025
@vramik
Copy link
Contributor
vramik commented Apr 10, 2025

@edewit GHA failed for this one, not sure if it's related or not, could you take a look?

Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>
@edewit
Copy link
Contributor Author
edewit commented Apr 10, 2025

yeap, fixed it

@vramik vramik requested a review from ssilvert April 10, 2025 18:59
Copy link
Contributor
@ssilvert ssilvert left a comment

Choose a reason for hiding this comment

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

LGTM

@vramik
Copy link
Contributor
vramik commented Apr 10, 2025

Tested locally and it works fine.

@ssilvert ssilvert merged commit 514b1b4 into keycloak:main Apr 11, 2025
55 checks passed
@edewit edewit deleted the issue-38767 branch April 16, 2025 07:36
edewit added a commit to edewit/keycloak that referenced this pull request Apr 22, 2025
…38768)

* make group select required when specific group is selected

fixes: keycloak#38767
Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>

* small refactor to make labels not dependant on route

Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>

* fixed tests

Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>

---------

Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>
(cherry picked from commit 514b1b4)
stianst pushed a commit that referenced this pull request Apr 23, 2025
…39114)

* make group select required when specific group is selected

fixes: #38767
Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>

* small refactor to make labels not dependant on route

Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>

* fixed tests

Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>

---------

Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>
(cherry picked from commit 514b1b4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make group required when selecting a specific group creating a premission
3 participants
0