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.

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

V.25.0.6 #10

Open
wants to merge 12 commits into
base: 20.0.1
Choose a base branch
from
Open

V.25.0.6 #10

wants to merge 12 commits into from

Conversation

Tobasco99
Copy link

@Tobasco99 Tobasco99 commented Sep 26, 2024

Previous Attempt

This version of the keycloak extension was created the following:

  1. Tried to follow the described workflow in this repositories wiki. Note: the link to the Account console source files is not working anymore as v2 was removed in version 24.0.0.
  2. Try to adjust the previous version of the account console by following the instructions of the official changelog. This included adjusting the content.json and the parents.
  3. Try to build the project by executing npm build. Once all occuring errors got resolved, I build the jars.

Potential attempt to solve the issues

In order to resolve the errors, one can try the following:

  1. Rewrite the current custom files mentioned in the wiki to a project structure like described here. This includes adjusting all imports (because of project structure change), as well as updating all versions of the used libaries. Make sure to adjust the tsconfig accordingly.
  2. Once a potential version is build, you can test it by deploying a local keycloak instance and move the jars to the providers folder.

Also note that once the version is updated, the learning-layers theme needs to be updated as well. For this exchange the value in login/theme.properties to:
stylesCommon=node_modules/@patternfly/patternfly/patternfly.min.css node_modules/patternfly/dist/css/patternfly.min.css node_modules/patternfly/dist/css/patternfly-additions.min.css lib/pficon/pficon.css

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