move themes custom authentication titles to dedicated block (#65803) #20

Merged
smihai merged 2 commits from wip/65803-misc-move-custom-authentication-titles-to-dedicated-block into main 2023-03-30 09:04:57 +02:00
Owner
No description provided.
smihai force-pushed wip/65803-misc-move-custom-authentication-titles-to-dedicated-block from ec879efb99 to 585d535df1 2022-11-23 21:33:24 +01:00 Compare
tjund requested review from tjund 2022-11-24 11:12:22 +01:00
smihai force-pushed wip/65803-misc-move-custom-authentication-titles-to-dedicated-block from 4cfce8a312 to 2171ff7aef 2022-11-25 17:30:20 +01:00 Compare
Owner

Je suis repassé sur les thèmes.

Je suis repassé sur les thèmes.
Author
Owner

Je pense inutile de gérer templates/variants/metz-metropole-2019/authentic2_auth_saml/login.html. Il faudrait le virer sur l'exemple de la PR#55.

Je pense inutile de gérer templates/variants/metz-metropole-2019/authentic2_auth_saml/login.html. Il faudrait le virer sur l'exemple de la PR#55.
Owner

Et le template saml icitoyen ?

Et le template saml icitoyen ?
Author
Owner

Et le template saml icitoyen ?

Je pense qu'il faut garder celui là car il pose un titre h2.

> Et le template saml icitoyen ? Je pense qu'il faut garder celui là car il pose un titre h2.
Owner

J'ai fait un tour des themes imio.
Pour moi ce patch n'a aucune influence sur les thèmes imio car ils surchargent sans étendre les templates registration/registration_form.html et authentic2/ login.html, et login_password_registration_form.html.

@Serghei: Mais je veux bien ton regard.

J'ai fait un tour des themes imio. Pour moi ce patch n'a aucune influence sur les thèmes imio car ils surchargent sans étendre les templates `registration/registration_form.html` et `authentic2/ login.html`, et `login_password_registration_form.html`. @Serghei: Mais je veux bien ton regard.
Owner

Je pense inutile de gérer templates/variants/metz-metropole-2019/authentic2_auth_saml/login.html.

Template supprimé

> Je pense inutile de gérer templates/variants/metz-metropole-2019/authentic2_auth_saml/login.html. Template supprimé
Author
Owner

J'ai fait un tour des themes imio.
Pour moi ce patch n'a aucune influence sur les thèmes imio car ils surchargent sans étendre les templates registration/registration_form.html et authentic2/ login.html, et login_password_registration_form.html.

Je suis tout à fait d'accord.

Mais l'idée serait quand même que dans https://git.entrouvert.org/imio-publik-themes.git/tree/templates/authentic2/login.html on n'ait pas à faire:

{% elif id == "password" %}
<h2>Connexion avec identifiant</h2>
{% elif id == "fedict" %}
<h2>Connexion avec carte d’identité</h2>
{% else %}
<!-- no title for oidc -->
{% endif %}

mais rajouter les titres dans les templates pour chaque moyen d'authentification.

> J'ai fait un tour des themes imio. > Pour moi ce patch n'a aucune influence sur les thèmes imio car ils surchargent sans étendre les templates `registration/registration_form.html` et `authentic2/ login.html`, et `login_password_registration_form.html`. > Je suis tout à fait d'accord. Mais l'idée serait quand même que dans https://git.entrouvert.org/imio-publik-themes.git/tree/templates/authentic2/login.html on n'ait pas à faire: ``` {% elif id == "password" %} <h2>Connexion avec identifiant</h2> {% elif id == "fedict" %} <h2>Connexion avec carte d’identité</h2> {% else %} <!-- no title for oidc --> {% endif %} ``` mais rajouter les titres dans les templates pour chaque moyen d'authentification.
pmarillonnet requested changes 2023-03-02 11:12:48 +01:00
pmarillonnet left a comment
Owner

Dans #53264 qui est le complémentaire côté a2, on dégage le bloc before-login du gabarit auth_saml. Je vois bien la répercussion sur le gabarit redéfini pour le thème de Roanne ici, mais à taper un git grep before-login dans PBT j’en vois trois autres qui n’apparaissent pas ici. Je pense qu’il faut faire la même modification que pour Roanne.

Dans #53264 qui est le complémentaire côté a2, on dégage le bloc `before-login` du gabarit auth_saml. Je vois bien la répercussion sur le gabarit redéfini pour le thème de Roanne ici, mais à taper un `git grep before-login` dans PBT j’en vois trois autres qui n’apparaissent pas ici. Je pense qu’il faut faire la même modification que pour Roanne.
Author
Owner

Dans #53264 qui est le complémentaire côté a2, on dégage le bloc before-login du gabarit auth_saml.

Finalement le bloc n'est pas dégagé, mais déplacé dans {% block content %}

> Dans #53264 qui est le complémentaire côté a2, on dégage le bloc `before-login` du gabarit auth_saml. Finalement le bloc n'est pas dégagé, mais déplacé dans `{% block content %}`
smihai changed title from WIP: move themes custom authentication titles to dedicated block (#65803) to move themes custom authentication titles to dedicated block (#65803) 2023-03-09 16:50:39 +01:00
pmarillonnet approved these changes 2023-03-29 15:12:51 +02:00
pmarillonnet left a comment
Owner

C’est bon pour moi, à déployer assez tôt de pair avec #53264 pour profiter du cycle de trois semaines et voir arriver en recettes d’éventuels glitches etc.

C’est bon pour moi, à déployer assez tôt de pair avec #53264 pour profiter du cycle de trois semaines et voir arriver en recettes d’éventuels glitches etc.
smihai force-pushed wip/65803-misc-move-custom-authentication-titles-to-dedicated-block from a0a71a03b8 to af836b4fc6 2023-03-30 08:37:13 +02:00 Compare
smihai force-pushed wip/65803-misc-move-custom-authentication-titles-to-dedicated-block from af836b4fc6 to 141221251e 2023-03-30 09:03:30 +02:00 Compare
smihai merged commit 141221251e into main 2023-03-30 09:04:57 +02:00
smihai deleted branch wip/65803-misc-move-custom-authentication-titles-to-dedicated-block 2023-03-30 09:04:57 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#20
No description provided.