entrouvert
/
authentic
16
0
Fork 0
Code Issues Pull Requests 32 Releases Activity

misc: render authenticators names in their own templates (#53264) #5

Merged
smihai merged 2 commits from wip/53264-move-authenticator-name-in-own-template into main 2023-03-28 13:26:42 +02:00
Conversation 3 Commits 2 Files Changed 16 +149 -73
smihai commented 2023-03-01 16:37:35 +01:00
Owner
Copy Link
No description provided.
smihai force-pushed wip/53264-move-authenticator-name-in-own-template from 7878c4b82b to 25ae517ca7 2023-03-01 18:25:42 +01:00 Compare
smihai force-pushed wip/53264-move-authenticator-name-in-own-template from 25ae517ca7 to 90747b5b1a 2023-03-02 09:24:30 +01:00 Compare
pmarillonnet reviewed 2023-03-02 10:46:18 +01:00
src/authentic2_auth_fc/views.py
@ -78,3 +78,3 @@
context['login_url'] = utils_misc.make_url('fc-login-or-link', keep_params=True, request=request)
context['block-extra-css-class'] = 'fc-login'
template = 'authentic2_auth_fc/login.html'
template = kwargs.get('template', 'authentic2_auth_fc/login.html')
pmarillonnet commented 2023-03-02 10:46:18 +01:00
Owner
Copy Link

Est-ce que tu peux m’expliquer à quel moment on a besoin d’avoir ça stp ? J’ai loupé dans le reste du patch le moment qui justifiait d’avoir ce nom de template qui peut varier, avec le fallback sur la valeur qui était en dur dans le code jusque là.

Est-ce que tu peux m’expliquer à quel moment on a besoin d’avoir ça stp ? J’ai loupé dans le reste du patch le moment qui justifiait d’avoir ce nom de template qui peut varier, avec le fallback sur la valeur qui était en dur dans le code jusque là.
smihai commented 2023-03-08 12:38:40 +01:00
Author
Owner
Copy Link

L'idée ici est de rajouter un template pour la page d'inscription, qui permettrait de définir le titre du bloc FC.

L'idée ici est de rajouter un template pour la page d'inscription, qui permettrait de définir le titre du bloc FC.
pmarillonnet requested changes 2023-03-02 11:09:14 +01:00
pmarillonnet left a comment
Owner
Copy Link

Outre les commentaires ciblés au dessus et ci-dessous, de façon plus générale je trouve le patche trop costaud pour passer sans tests. Est-ce qu’il serait stp possible d’en mettre un pour la page de login et un pour la page d’enregistrement, avec si possible une redéfinition de gabarit local à un des moyens d’auth activés, pour prouver qu’on a bien le comportement désiré de redéfinition du titre géré par le gabarit propre au moyen d’authn ?

Outre les commentaires ciblés au dessus et ci-dessous, de façon plus générale je trouve le patche trop costaud pour passer sans tests. Est-ce qu’il serait stp possible d’en mettre un pour la page de login et un pour la page d’enregistrement, avec si possible une redéfinition de gabarit local à un des moyens d’auth activés, pour prouver qu’on a bien le comportement désiré de redéfinition du titre géré par le gabarit propre au moyen d’authn ?
src/authentic2_auth_fc/templates/authentic2_auth_fc/login.html
@ -12,0 +17,4 @@
</a>
</div>
</div>
<div id="fc-explanation-link">
pmarillonnet commented 2023-03-02 10:54:41 +01:00
Owner
Copy Link

Clairement il y a eu un loupé ici, la div fc-explanation-link est déjà présente (et inchangée) quelques lignes plus bas dans ce même gabarit.

Clairement il y a eu un loupé ici, la div `fc-explanation-link` est déjà présente (et inchangée) quelques lignes plus bas dans ce même gabarit.
smihai commented 2023-03-08 12:39:07 +01:00
Author
Owner
Copy Link

En effet, je viens de corriger et pousser la branche.

En effet, je viens de corriger et pousser la branche.
src/authentic2_auth_oidc/templates/authentic2_auth_oidc/login.html
@ -1,11 +1,12 @@
{% block login %}
<div>
pmarillonnet commented 2023-03-02 10:48:48 +01:00
Owner
Copy Link

Ici pourquoi juste on ajoute une div ? Pas compris ce que la modification de ce fichier vient faire là, mais je loupe peut-

Ici pourquoi juste on ajoute une div ? Pas compris ce que la modification de ce fichier vient faire là, mais je loupe peut-
smihai commented 2023-03-08 12:40:30 +01:00
Author
Owner
Copy Link

C'est la suggestion (juste) de @tjund pour uniformiser la structure DOM de chaque bloc d'authentification.

C'est la suggestion (juste) de @tjund pour uniformiser la structure DOM de chaque bloc d'authentification.
src/authentic2_auth_saml/templates/authentic2_auth_saml/login.html
@ -1,21 +1,28 @@
{% load i18n %}
{% block before-login %}
pmarillonnet commented 2023-03-02 10:52:49 +01:00
Owner
Copy Link

Ok mais ici dans le ticket PBT lié on ne traite pas toutes les occurrences de surcharge du bloc before-content il me semble. Je vais coller la liste dans la PR PBT liée.

Ok mais ici dans le ticket PBT lié on ne traite pas toutes les occurrences de surcharge du bloc `before-content` il me semble. Je vais coller la liste dans la PR PBT liée.
smihai force-pushed wip/53264-move-authenticator-name-in-own-template from 90747b5b1a to 2c4a240d44 2023-03-08 12:33:53 +01:00 Compare
pmarillonnet commented 2023-03-09 08:25:21 +01:00
Owner
Copy Link

À la lecture de tes réponses, je n’ai pas compris si la PR est de nouveau bonne pour relecture ? On dirait que oui mais je n’ai pas été requis pour une nouvelle relecture et le ticket lié est toujours en “Solution proposée”.

À la lecture de tes réponses, je n’ai pas compris si la PR est de nouveau bonne pour relecture ? On dirait que oui mais je n’ai pas été requis pour une nouvelle relecture et le ticket lié est toujours en “Solution proposée”.
smihai requested review from pmarillonnet 2023-03-09 16:31:03 +01:00
pmarillonnet commented 2023-03-21 16:23:11 +01:00
Owner
Copy Link

Outre les commentaires ciblés au dessus et ci-dessous, de façon plus générale je trouve le patche trop costaud pour passer sans tests. Est-ce qu’il serait stp possible d’en mettre un pour la page de login et un pour la page d’enregistrement, avec si possible une redéfinition de gabarit local à un des moyens d’auth activés, pour prouver qu’on a bien le comportement désiré de redéfinition du titre géré par le gabarit propre au moyen d’authn ?

Cette remarque s’est perdue dans la masse, il me semble :)

> Outre les commentaires ciblés au dessus et ci-dessous, de façon plus générale je trouve le patche trop costaud pour passer sans tests. Est-ce qu’il serait stp possible d’en mettre un pour la page de login et un pour la page d’enregistrement, avec si possible une redéfinition de gabarit local à un des moyens d’auth activés, pour prouver qu’on a bien le comportement désiré de redéfinition du titre géré par le gabarit propre au moyen d’authn ? Cette remarque s’est perdue dans la masse, il me semble :)
smihai force-pushed wip/53264-move-authenticator-name-in-own-template from 2c4a240d44 to b2d21d0832 2023-03-21 17:30:44 +01:00 Compare
smihai force-pushed wip/53264-move-authenticator-name-in-own-template from b2d21d0832 to 5340d7f6a8 2023-03-27 01:05:20 +02:00 Compare
smihai force-pushed wip/53264-move-authenticator-name-in-own-template from 5340d7f6a8 to 238575669b 2023-03-27 10:37:03 +02:00 Compare
smihai commented 2023-03-27 10:40:56 +02:00
Author
Owner
Copy Link

Cette remarque s’est perdue dans la masse, il me semble :)

Oui, mes excuses. Tests rajoutés.

> > Cette remarque s’est perdue dans la masse, il me semble :) > Oui, mes excuses. Tests rajoutés.
pmarillonnet approved these changes 2023-03-28 13:25:44 +02:00
pmarillonnet left a comment
Owner
Copy Link

Ok pour moi.

Ok pour moi.
smihai merged commit 4c02778eb9 into main 2023-03-28 13:26:42 +02:00
smihai deleted branch wip/53264-move-authenticator-name-in-own-template 2023-03-28 13:26:42 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 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/authentic#5
No description provided.
Delete Branch "wip/53264-move-authenticator-name-in-own-template"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?