Development #45507
ne plus charger __services.js en front
0%
Description
Il est utile en backoffice pour avoir les services pour la construction du menu publik mais en front il sert juste à fournir quelques variables js (PUBLIK_PORTAL_AGENT_URL et PUBLIK_PORTAL_AGENT_TITLE), elles pourraient tout aussi bien être posées directement dans l'HTML.
(surtout, ça permettra d'annuler la correction de #30488, refaire en sorte que __services.js déclenche un SSO, ce qui servira pour permettre la construction du menu Publik via des pages du portail agent).
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Fichier 0001-templates-don-t-load-__services.js-in-front-anymore-.patch 0001-templates-don-t-load-__services.js-in-front-anymore-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
(ça cassera static/pratic/extra.js qui exploite COMBO_KNOWN_SERVICES pour trouver l'url de l'authentic mais c'est sur un déploiement plus maintenu et si c'est à reprendre on y fera les choses différemment).
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Lié à Project management #41402: construire le menu latéral du portail agent à partir de pages combo ajouté
Mis à jour par Thomas Noël il y a plus de 3 ans
J'ai l'impression qu'il va manquer notammement PUBLIK_ENVIRONMENT_LABEL (égal à {{environment_label}}) qui est nécessaire en front pour dessiner le «bandeau du coin» en front ?
Mis à jour par Frédéric Péters il y a plus de 3 ans
Il est posé via la variable django existant dans le contexte, pas via js,
{% if environment_label %}<span id="environment-label">{{ environment_label }}</span>{% endif %}
Mis à jour par Thomas Noël il y a plus de 3 ans
J'ai été enduit d'erreur par ces lignes dans static/portal-agent/js/publik.js :
if (PUBLIK_ENVIRONMENT_LABEL) { $('body').attr('data-environment-label', PUBLIK_ENVIRONMENT_LABEL); }
Donc si je comprends, en front c'est via templates/theme.html mais c'est écrasé par static/portal-agent/js/publik.js + __services.js pour le portail agent ?
(Je valide de toute façon... même pas vraiment la peine de répondre à ma question)
Mis à jour par Thomas Noël il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 3 ans
Donc si je comprends, en front c'est via templates/theme.html mais c'est écrasé par static/portal-agent/js/publik.js + __services.js pour le portail agent ?
pour le portail agent et tous les backoffices, pour taper le bandeau "travaux"; à noter que ça devrait pouvoir être dégagé et directement intégré dans le gadjo/base.html de publik-base-theme (ce n'était pas ainsi de l'époque où le backoffice de wcs ne passait pas par là, je pense).
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 0c8989441e99f373a77333acf659ee5a4dbe9a7f Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Jul 27 21:34:16 2020 +0200 templates: don't load __services.js in front anymore (#45507)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
templates: don't load __services.js in front anymore (#45507)