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

@keycloakify/angular #5

Closed
wants to merge 64 commits into from
Closed

@keycloakify/angular #5

wants to merge 64 commits into from

Conversation

garronej
Copy link
Collaborator

No description provided.

@luca-peruzzo
Copy link
Collaborator

luca-peruzzo commented Sep 30, 2024

Task list

  • absolute/relative imports
  • import type
  • i18n rework
  • account theme
  • user profile form field rework
  • template rework ?

@luca-peruzzo
Copy link
Collaborator

@kathari00 @garronej here the task list about requested changes

@garronej
Copy link
Collaborator Author

Great thank you luca, looking great.

@luca-peruzzo
Copy link
Collaborator

@garronej @kathari00 all tasks are completed (the question of the template remains).
due to the changes now in starters we have to do something like that depending on the presence of the account theme:

          window.kcContext?.themeType === 'account'
            ? provideKeycloakifyAngularAccount({
                classes,
                doUseDefaultCss,
                getI18n: getI18n,
              })
            : provideKeycloakifyAngularLogin({
                classes,
                doMakeUserConfirmPassword,
                doUseDefaultCss,
                getI18n: getI18n,
              }),

@garronej is it something you can handle when someone initializes the account theme?

@garronej garronej closed this Oct 2, 2024
@luca-peruzzo luca-peruzzo mentioned this pull request Oct 4, 2024
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.

2 participants