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

feat: upgrade to Keycloakify v5 #29

Merged
merged 1 commit into from Jun 25, 2022

Conversation

garronej
Copy link
Contributor

@garronej garronej commented Jun 22, 2022

Hi @revolunet,

Following up on our discussion here is a PR to upgrade to Keycloakify v5.

image

I should mention that it's now better to overload register-user-profile.ftl rather than register.ftl, Keycloak didn't remove register.ftl only for backward compatibility but it can now be considered legacy. See this https://docs.keycloakify.dev/realtime-input-validation

Best regards

@revolunet
Copy link
Member

thanks a lot for your help !

@revolunet revolunet requested a review from maxgfr June 23, 2022 11:11
@revolunet
Copy link
Member

will rebase #28 once merged

@revolunet revolunet changed the title Upgrade to Keycloakify v5 feat: upgrade to Keycloakify v5 Jun 23, 2022
Copy link
Member

@maxgfr maxgfr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice 😎

@revolunet revolunet merged commit e76c831 into SocialGouv:main Jun 25, 2022
@garronej garronej deleted the upgrade_to_keycloakify_v5 branch June 25, 2022 16:38
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.

None yet

3 participants