Project

General

Profile

Development #45507

ne plus charger __services.js en front

Added by Frédéric Péters over 3 years ago. Updated over 3 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
27 July 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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).


Files


Related issues

Related to Publik - Project management #41402: construire le menu latéral du portail agent à partir de pages comboFermé07 April 2020

Actions

Associated revisions

Revision 0c898944 (diff)
Added by Frédéric Péters over 3 years ago

templates: don't load __services.js in front anymore (#45507)

History

#1

Updated by Frédéric Péters over 3 years ago

(ç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).

#2

Updated by Frédéric Péters over 3 years ago

#3

Updated by Thomas Noël over 3 years ago

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 ?

#4

Updated by Frédéric Péters over 3 years ago

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 %}
#5

Updated by Thomas Noël over 3 years ago

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)

#6

Updated by Thomas Noël over 3 years ago

  • Status changed from Solution proposée to Solution validée
#7

Updated by Frédéric Péters over 3 years ago

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).

#8

Updated by Frédéric Péters over 3 years ago

  • Status changed from Solution validée to 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)
#9

Updated by Frédéric Péters over 3 years ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF