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.

Sign up for GitHub

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

Jump to bottom

Immediately reflect flow binding status on flow definition page in Admin UI when binding an auth flow #26636

Closed
thomasdarimont opened this issue Jan 31, 2024 · 1 comment · Fixed by #26685
Closed

Immediately reflect flow binding status on flow definition page in Admin UI when binding an auth flow #26636

thomasdarimont opened this issue Jan 31, 2024 · 1 comment · Fixed by #26685
Labels
area/admin/ui good first issue help wanted impact/low kind/bug Categorizes a PR related to a bug release/24.0.0 team/ui

Comments

@thomasdarimont
Copy link
Contributor

Description

Currently when binding a custom flow from within the flow definition page the flow binding status is not reflected directly.

Users have to leave the flow definition page and return to see the effect.

image image

Discussion

No response

Motivation

No response

Details

Binding the flow should update the flow status immediately.
@thomasdarimont thomasdarimont added kind/enhancement Categorizes a PR related to an enhancement status/triage labels Jan 31, 2024
@jonkoops jonkoops added kind/bug Categorizes a PR related to a bug area/admin/ui team/ui and removed kind/enhancement Categorizes a PR related to an enhancement labels Jan 31, 2024
@ssilvert
Copy link
Contributor

I have recreated this. I agree with @jonkoops that it's a bug. Low priority, but we should fix it.

edewit added a commit to edewit/keycloak that referenced this issue Feb 1, 2024
@edewit
fix flow status
f31d888 
fixes: keycloak#26636

Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>
@edewit edewit mentioned this issue Feb 1, 2024
Merged
@jonkoops jonkoops added this to the Backlog milestone Feb 1, 2024
edewit added a commit that referenced this issue Feb 12, 2024
@edewit
fix flow status (#26685)
39bd59c 
fixes: #26636

Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>
@ahus1 ahus1 removed this from the Backlog milestone Mar 6, 2024
ahus1 pushed a commit to ahus1/keycloak that referenced this issue Mar 22, 2024
@edewit @ahus1
fix flow status (keycloak#26685)
93d3c6c 
fixes: keycloak#26636

Signed-off-by: Erik Jan de Wit <erikjan.dewit@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/admin/ui good first issue help wanted impact/low kind/bug Categorizes a PR related to a bug release/24.0.0 team/ui
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix flow status edewit/keycloak
4 participants
@thomasdarimont @ssilvert @jonkoops @ahus1