Produits Entr'ouvert » Intégrations graphiques Publik

Lauréline Guerin a écrit :

(Je ne suis pas sûre de la durée du cache de chaque cellule)

10 secondes ça ne me semble pas utile, c'est en gros le temps de lire la page... donc le prochain affichage ne profitera pas du cache.

Mais si on mets plus, ça ne va pas non plus : si on arrive sur cette page après un formulaire qui aura modifié des informations, il faut que ça soit vu aussitôt. Idem si on imagine la situation d'un agent en train de faire une modif dans axel avec l'usager au bout du fil qui vérifie "en temps réel avec F5F5F5F5" que son enfant a bien été déclaré absent.

Bref, je ne mettrais pas de cache, ça évite absolument tout problème d'invalidation de cache :) Et si un jour on se rend compte que ça tape trop, on en mettra.

Sur les "name" je serais pour préfixer avec quelque chose comme "DUI (Teamnet Axel) : " et mettre des trucs plus explicites, genre :

"name": "DUI (Teamnet Axel) : Activités auxquelles l'enfant est inscrit·e" 
          "name": "DUI (Teamnet Axel) : Informations générales sur l'enfant",
          "name": "DUI (Teamnet Axel) : Liste des enfants",

etc.

Et le plus important : dans les URL il faut remplacer "...toulouse-axel/test/..." par "...toulouse-axel/{{ slug }}/..." et ajouter sur chaque cellule :

          "form": [
            {"label": "Identifiant du connecteur", "varname": "slug", "required": true}
          ],

Ainsi, quand une cellule sera ajoutée, l'admin fonctionnel devra y configurer le nom du slug.

Ah oui aussi : c'est "JSON_CELL_TYPES.update" et pas juste "JSON_CELL_TYPES". Le ".update" est un petit sucre syntaxique qui permet de faire comprendre à Combo que le dictionnaire settings.JSON_CELL_TYPES doit subir un update, et non pas être complétement écrasé.

Thomas Noël a écrit :

Ah oui aussi : c'est "JSON_CELL_TYPES.update" et pas juste "JSON_CELL_TYPES". Le ".update" est un petit sucre syntaxique qui permet de faire comprendre à Combo que le dictionnaire settings.JSON_CELL_TYPES doit subir un update, et non pas être complétement écrasé.

Il n'y a que 4 cas d'usage¹, toujours les mêmes, je me dis avec le recul qu'on pourrait apprendre au loader des thème de faire un update systématiquement pour ces settings, ça éviterait des erreurs.

¹

publik-base-theme$ git grep -h "COMBO_ASSET_SLOTS\|COMBO_CELL_ASSET_SLOTS\|COMBO_PUBLIC_TEMPLATES\|JSON_CELL_TYPES" \*.json | sed 's/^ *//' | sort -u
"COMBO_ASSET_SLOTS.update": {
"COMBO_CELL_ASSET_SLOTS.update": {
"COMBO_PUBLIC_TEMPLATES.update": {
"JSON_CELL_TYPES.update": {

Je me dis avec le recul qu'on pourrait apprendre au loader des thème de faire un update systématiquement pour ces settings.

Non, il pourrait arriver une situation où on voudrait repartir de zéro pour COMBO_CELL_ASSET_SLOTS et COMBO_PUBLIC_TEMPLATES; je préfère ne pas introduire de comportement particulier.

Par contre on a déjà, dans publik-base-theme, une vérification des clés, qui dans sa liste actuelle va interrompre le build si JSON_CELL_TYPES est utilisé :

E: unknown settings key for $theme (JSON_CELL_TYPES)

(mais jenkins ne passe par là que lors du build du paquet)