Bug #25524
mellon/base.html vs combo et les autres applications
0%
Description
mellon/base.html fait maintenant {% extends theme_base %} (#25520) mais si c'est très bien quand l'origine est fargo, ça foire quand l'origine est combo (qui n'a pas de theme_base parce qu'il n'utilise pas l'infra de partage de thème).
Je vois deux options possibles, #25523 pour simplement se passer de mellon/base.html, en utilisant django.contrib.messages (ça a ma préférence), ou ajouter dans fargo et authentic2-auth-saml un mellon/base.html pointant vers le base.html de ces applications.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-templates-do-not-ship-mellon-base.html-as-it-can-t-b.patch 0001-templates-do-not-ship-mellon-base.html-as-it-can-t-b.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mais django.contrib.messages çe le fait uniquement, parfois, côté portail usager, pas dans les autres applications, qui n'ont pas de page publique où afficher le message.
Du coup, retrait ici du mellon/base.html qui ne peut pas être partagé, aux applications à assurer leurs pages.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Lié à Bug #25526: template pour les pages d'erreur de django-mellon ajouté
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Lié à Bug #25528: template pour les pages d'erreur de django-mellon ajouté
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Lié à Bug #25529: template pour les pages d'erreur de django-mellon ajouté
Mis à jour par Frédéric Péters il y a plus de 5 ans
C'est un peu lourd de devoir ainsi taper des templates partout mais ça me semble quand même plus net ainsi, avec l'avantage de rester dans le style "gadjo" dans les applications qui ne font que du backoffice (passerelle, chrono, mais aussi bijoe & corbo pour lesquels il n'y apas encore fait de patchs).
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Pas possible de diffuser dans les contextes de template de combo un theme_base qui soit theme.html dans combo ? (et pas un template chargé dynamiquement) ? Il me semble que ça simplifierait la définition des base.html pour n'importe quelle application.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Pas possible de diffuser dans les contextes de template de combo un theme_base qui soit theme.html dans combo ? (et pas un template chargé dynamiquement) ? Il me semble que ça simplifierait la définition des base.html pour n'importe quelle application.
Ça ne m'a pas semblé évident; et comme je souhaitais en plus en profiter pour que les applications backoffice utilisent gadjo pour ces pages, pas terriblement utile. Aussi, fournir mellon/base.html dans publik-base-html reste de toute façon une mauvaise idée parce que ça le place avant un mellon/base.html qui serait défini dans une application, qui pourrait avoir ses spécificités (genre il pourrait y avoir un sens à ce qu'il existe dans authentic2-auth-saml, adapté au contexte proxying idp).
Mis à jour par Frédéric Péters il y a plus de 5 ans
(J'aimerais bien qu'on arrive à taper une correction dans cette release)
Mis à jour par Frédéric Péters il y a plus de 5 ans
(J'aimerais bien qu'on arrive à taper une correction dans cette release)
Toujours le cas; pour récapituler il s'agit ici de supprimer de publik-base-theme django-mellon/base.html pour permettre aux applications de fournir ce template, et d'y utiliser un {% extends "..." %} adéquat.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Ok, pas fan du remède mais je ne vois rien d'autre pour aller vers ce que tu veux.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4da715f79d060707c06596a26687403ac3872959 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Jul 30 07:31:36 2018 +0200 templates: do not ship mellon/base.html as it can't be shared (#25524) combo and other apps expects a different {% extends ... %} value.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Tu peux aussi regarder/valider les trois tickets liés qui ajoutent les fichiers concernés dans les applications ?
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
templates: do not ship mellon/base.html as it can't be shared (#25524)
combo and other apps expects a different {% extends ... %} value.