8000 Add form revert logic for tab providers by uzmamansoor09 · Pull Request #49331 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content

Add form revert logic for tab providers - #49331

Merged
rmartinc merged 2 commits into
keycloak:mainfrom
uzmamansoor09:47432
May 28, 2026
Merged

rmartinc merged 2 commits into
keycloak:mainfrom
uzmamansoor09:47432

Conversation

@uzmamansoor09
Copy link
Copy Markdown
Contributor

Closes #47432.
Fixes an issue in the admin UI custom tab extension where clicking the cancel button triggers an incorrect redirection.

  • If the provider is a PAGE_PROVIDER, the "Cancel" button (which redirects the user) is displayed.
  • If the provider is a TAB_PROVIDER, a "Revert" button is displayed instead, which resets the form state without redirecting.

@rmartinc rmartinc 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.

Thanks @uzmamansoor09! LGTM, just check if you need the data-testid attribute in the cancel/revert buttons for the follow-up test in the quickstarts. If you need the attribute you can add it before here.

@uzmamansoor09
uzmamansoor09 marked this pull request as ready for review May 27, 2026 09:43
@uzmamansoor09
uzmamansoor09 force-pushed the 47432 branch 3 times, most recently from 734d156 to d5d1108 Compare May 27, 2026 10:34
…tab extension

Signed-off-by: uzmamansoor09 <Uzma.Mansoor@ibm.com>
Signed-off-by: uzmamansoor09 <Uzma.Mansoor@ibm.com>
@rmartinc rmartinc 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.

Thanks @uzmamansoor09! LGTM!

@rmartinc rmartinc self-assigned this May 28, 2026
@rmartinc
rmartinc merged commit 31e4e8c into keycloak:main May 28, 2026
66 checks passed
@stianst stianst mentioned this pull request Jun 1, 2026
sashyo pushed a commit to tide-foundation/keycloak-IGA that referenced this pull request Jul 20, 2026
* Closes keycloak#47432. Reset form on cancel button click in admin ui tab extension

Signed-off-by: uzmamansoor09 <Uzma.Mansoor@ibm.com>

* Following up review

Signed-off-by: uzmamansoor09 <Uzma.Mansoor@ibm.com>

---------

Signed-off-by: uzmamansoor09 <Uzma.Mansoor@ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The cancel button in the custom UiTabProvider encounters an issue where it redirects to an incorrect URL and triggers an error.

3 participants

0