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: add logout-confirm #120

Merged
merged 3 commits into from Jul 7, 2022
Merged

feat: add logout-confirm #120

merged 3 commits into from Jul 7, 2022

Conversation

revolunet
Copy link
Collaborator

Bonjour, pour Keycloak 18, je souhaiterai ajouter la possibilité de thémer
logout-confirm.tfl

La traduction française des nouveaux termes n'est pas encore dispo, y-a-t-il moyen de fournir une trad par défaut en FR pour les nouveaux termes (ex: logoutConfirmTitle ou logoutConfirmHeader), ou faut-il générer les fichiers pour la
18.0.0 ?

garronej added a commit that referenced this pull request Jun 16, 2022
@garronej
Copy link
Collaborator

Salut @revolunet !!
Génial pour la PR! Hâte de merger,
J'ai mis à jour les ressources i18n, ce sont maintenant celle de Keycloak 18.0.1.

Pour ce qui est de la langue, c'est toi qui est sensé la passer dans l'url de redirection au login:

import keycloak_js from "keycloak-js";
import { createKeycloakAdapter } from "keycloakify";
import { addParamToUrl } from "powerhooks/tools/urlSearchParams";

//...

const keycloakInstance = keycloak_js({
    "url": "http://keycloak-server/auth",
    "realm": "myrealm",
    "clientId": "myapp",
});

keycloakInstance.init({
    "onLoad": "check-sso",
    "silentCheckSsoRedirectUri": window.location.origin + "/silent-check-sso.html",
    "adapter": createKeycloakAdapter({
        "transformUrlBeforeRedirect": url=>
            [url]
                .map(url => addParamToUrl({ url, "name": "ui_locales", "value": "fr" }).newUrl)
                [0],
        keycloakInstance,
    }),
});

sinon tu peux régler la langue par défaut dans la config de keycloak:

image

Jspr que ça aide, fait moi savoir si tu as besoin.

À bientôt!

@revolunet
Copy link
Collaborator Author

revolunet commented Jun 17, 2022

Ca aide merci :) Mon keycloak est bien configuré en Français mais en fait ces nouvelles clés de traduction ne sont pas encore released en francais (alors que dispo dans la source). donc en attendant j'aimerai proposer la trad FR via mon thème si possible

@revolunet
Copy link
Collaborator Author

La doc "développement" me semble outdatée; comment je pourrai faire pour tester ?

@garronej
Copy link
Collaborator

C'est vrais qu'il n'y avais pas d'instruction dédier a la contribution.
Je viens d'en crée une.

@revolunet
Copy link
Collaborator Author

Ce serait possible de réparer la section "test with real keycloak instance" ? 🙏

@garronej
Copy link
Collaborator

Salut!
Ça fonctionne ça, je l'ai encore confirmé dimanche.
Il faut run yarn keycloak ou ˋnpx keycloakify`.
Tu as le detail de comment procéder qui est affiché en output.
Il faut que je fasses une video pour que ce soit plus clair

@garronej
Copy link
Collaborator

C'est bon tu l'as ou tu est bloqué ?

@revolunet
Copy link
Collaborator Author

Yes c'est bon yarn keycloak fonctionne bien dans mon projet merci, je teste et reviens vers toi asap

@revolunet
Copy link
Collaborator Author

revolunet commented Jun 21, 2022

une fois mvn installé, le yarn keycloak lancé, puis ./build_keycloak/start_keycloak_testing_container.sh

j'ai une page blanche une fois que je clique sur sign in
Capture d’écran 2022-06-21 à 16 44 00

si je remets le theme de base dans la conf keycloak, la page apparait bien

je ne sais pas si ca te parle

@garronej
Copy link
Collaborator

En général les erreur MIM types c'est qu'en fait c'est du HTML.
Je peut avoir le repo en question?

@revolunet
Copy link
Collaborator Author

Oui c'est ce thème : https://github.com/SocialGouv/keycloak-dsfr/tree/logout-confirm

Il était sur keycloakify@4 à la base j'ai du louper des étapes

@revolunet
Copy link
Collaborator Author

Looks good to me :)

@revolunet revolunet marked this pull request as ready for review July 7, 2022 10:37
@garronej garronej merged commit 00eab73 into keycloakify:main Jul 7, 2022
@garronej
Copy link
Collaborator

@all-contributors please add @revolunet for tests and code

@allcontributors
Copy link
Contributor

@garronej

I've put up a pull request to add @revolunet! 🎉

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

2 participants