8000 fix quick theme export to contain variables for both modes (#49125) by bu6n · Pull Request #49322 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content

fix quick theme export to contain variables for both modes (#49125) - #49322

Merged
edewit merged 1 commit into
keycloak:mainfrom
bu6n:fix-quick-theme-49125
May 26, 2026
Merged

edewit merged 1 commit into
keycloak:mainfrom
bu6n:fix-quick-theme-49125

Conversation

@bu6n
@bu6n bu6n commented May 26, 2026
Copy link
Copy Markdown
Contributor

This is a small fix for issue #49125.

I'm tagging @mabartos since I think you are the main person working on quick theme.

The code should speak for itself. One change I'm not sure is the one in PatternflyVars.ts. This was causing keys named "undefined" in the resulting theme JSON, but this might be used somewhere else?

I have used Claude to help me navigate the code, find the cause and write the fix. I think I understand what's happening here.

@bu6n
bu6n requested review from a team as code owners May 26, 2026 08:26
@edewit
edewit enabled auto-merge (squash) May 26, 2026 08:44
@mabartos
Copy link
Copy Markdown
Member

@bu6n Thanks for the PR! @edewit is the right person for these changes - thanks, Erik, for the review!

@edewit
edewit merged commit 3ed1822 into keycloak:main May 26, 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
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.

3 participants

0