Projet

Général

Profil

Development #40350

définir un template de base pour authentic2_auth_saml

Ajouté par Serghei Mihai il y a environ 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
02 mars 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Depuis #35083 on peut déclarer le template de base pour mellon.
Il faudrait s'en servir pour éviter les erreurs 500 lors d'un souci de SSO.


Fichiers


Demandes liées

Lié à django-mellon - Development #35083: Pouvoir indiquer un préfixe particulier pour les templates internes à mellonFermé26 juillet 2019

Actions

Révisions associées

Révision d55eddcd (diff)
Ajouté par Serghei Mihai il y a presque 4 ans

auth_saml: define templates base page (#40350)

Historique

#1

Mis à jour par Serghei Mihai il y a environ 4 ans

  • Lié à Development #35083: Pouvoir indiquer un préfixe particulier pour les templates internes à mellon ajouté
#2

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

  • Assigné à mis à Serghei Mihai
#3

Mis à jour par Serghei Mihai il y a presque 4 ans

#4

Mis à jour par Valentin Deniaud il y a presque 4 ans

Le patch ne s'applique pas, il a l'air basé sur des modifs que tu avais en local. Pousse une branche ?

#5

Mis à jour par Serghei Mihai il y a presque 4 ans

Yep, micmac chez moi en local.

Patch correct et branche poussée.

#6

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

C'est l'occasion d'ajouter un test qui échoue pour voir si le template est bien pris en compte et augmenter joyeusement la couverture du code, c'est dans tests/test_auth_saml.py.

#7

Mis à jour par Serghei Mihai il y a presque 4 ans

  • Statut changé de Solution proposée à En cours

Ouep, mais je doute fort que ça augmente la couverture du code d'authentic vu que l'argument s'applique à la vue de mellon.

#8

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

La couverture est une notion abstraite c'est pas seulement le chiffre renvoyé par coverage, faut juste voir qu'on arrive à traitre une erreur django-mellon sans planter sur le rendu du template.

#9

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

  • Statut changé de En cours à Solution validée

Allez c'est bon.

#10

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit e992567dab3256843e8876678beeb9bf0972d748
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Tue Jun 30 16:45:23 2020 +0200

    auth_saml: define templates base page (#40350)
#11

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF