Skip to content

Check the theme against the Keycloak it is written for - #37

Draft
Ekhorn wants to merge 1 commit into
28-migrate-kc-themes-to-vaadin-part-1from
28-migrate-kc-themes-to-vaadin-part-2
Draft

Ekhorn wants to merge 1 commit into
28-migrate-kc-themes-to-vaadin-part-1from
28-migrate-kc-themes-to-vaadin-part-2

Conversation

@Ekhorn

@Ekhorn Ekhorn commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

The pages here replace Keycloak's own, so the message keys they name and the set
of pages that exist are upstream's, and an upgrade can move either while
everything still compiles.

scripts/check-keycloak-drift.mjs fails on:

  • a page in src/pages that the Keycloak the Dockerfile pins no longer has
  • a message key the theme names that Keycloak no longer defines
  • a stand-in in i18n.ts that Keycloakify has caught up with and now ships

On a branch that moves the pinned version it also reports which of Keycloak's own
versions of our pages changed between the two. It keeps no baseline: a version's
templates never change, so the published jar for each version is the record.


Stack created with GitHub Stacks CLI • Give Feedback 💬

@Ekhorn
Ekhorn added this pull request to stack #41 October 2, 2026 13:24
The pages here replace Keycloak's own, so the message keys they name and the set
of pages that exist are upstream's, and an upgrade can move either while
everything still compiles.

scripts/check-keycloak-drift.mjs fails on:

- a page in src/pages that the Keycloak the Dockerfile pins no longer has
- a message key the theme names that Keycloak no longer defines
- a stand-in in i18n.ts that Keycloakify has caught up with and now ships

On a branch that moves the pinned version it also reports which of Keycloak's own
versions of our pages changed between the two. It keeps no baseline: a version's
templates never change, so the published jar for each version is the record.
@Ekhorn
Ekhorn force-pushed the 28-migrate-kc-themes-to-vaadin-part-2 branch from 655ef8c to b10b6f0 Compare October 2, 2026 14:01
@Ekhorn Ekhorn self-assigned this Oct 2, 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.

1 participant