Project

General

Profile

Development #28327

avoir une commande pour exporter des cellules JSON sous forme de thème

Added by Benjamin Dauvergne 7 months ago. Updated 7 months ago.

Status:
Nouveau
Priority:
Normal
Assignee:
-
Start date:
26 Nov 2018
Due date:
% Done:

0%

Patch proposed:
No
Planning:
No

Description

Pour simplifier le passage recette vers prod après prototypage.

History

#1 Updated by Frédéric Péters 7 months ago

Ça produirait un zip avec les fichiers templates bien nommés + des extraits de settings ?

#2 Updated by Benjamin Dauvergne 7 months ago

Frédéric Péters a écrit :

Ça produirait un zip avec les fichiers templates bien nommés + des extraits de settings ?

Il me manque vraiment d'habitude sur tout ça pour être précis, mais je viens de relire le patch sur les settings dans le thème et la nouvelle organisatoin de publik-base-theme en static/<theme>/config.json et donc je proposerai le principe suivant.

Avoir une commande:

produce-json-cells-for-theme <theme_name> <publik_base_theme_root_dir>

Pour le bon nommage, je me dis qu'utiliser la slugification du titre (qui n'est malheureusement pas obligatoire) est un bon début, mais je proposerai plutôt de demander systématiquement via un rawinput() de confirmer ce choix par défaut, voir d'ignorer complètement la cellule. De plus en cas d'utilisation de JSONCell il y a de grande chance qu'il y ait des doublons (qui sont évités avec JSON_CELL_TYPES) qu'il faudra détecter et ignorer.

Ça va ouvrir/créer static/<theme_name>/config.json et y ajouter pour chaque cellule la définition extraite de la cellule JSONCell, à coté de cela ça copiera le template dans templates/variants/<theme_name>/combo/cells/<json_cell_slug>.html. Je garderai peut-être la référence aux cellules d'origine dans la configuration du JSON_CELL_TYPES ('orig_cell_type_ids': [33, 45, 56]) pour les mises à jour.

Je ne pense pas que ça fera un boulot parfait, mais ça sera toujours plus simple qu'à la main il me semble (surtout les mises à jours).

#3 Updated by Benjamin Dauvergne 7 months ago

Aussi je pensais automatiquement faire correspondre les URLs comme https://passerelle.etc..<path> en [passerelle_url]<path>.

#5 Updated by Benjamin Dauvergne 7 months ago

Avis de Fred sur le sujet et ma réponse:

(17:16:50) /29758049991543413467430209: concernant le ticket combo export des cellules JSON, le plan te convient (https://dev.entrouvert.org/issues/28327) ou tu as une meilleure idée ?
(17:22:20) Fred Peters: Le fond du ticket, l'utilisation de json cell, m'ennuie parce que j'aimerais décourager cette pratique, et dans cette idée je trouve que ça fait beaucoup de taf d'automatisation pour ce qui devrait s'arrêter à trois copié/collés et le code posé dans le git et géré là. Mais une fois passé ce fond je n'ai pas d'objection.
(17:54:49) /29758049991543413467430209: dans la pratique ça voudrait dire que je m'occupe de l'intégration, c'est plus une question d'organisation du travail finalement
(17:56:46) /29758049991543413467430209: je vais déjà voir pour le volume

Pour moi ça pose la question de qui prototype et comment on prototype sur un projet pour moi et Mike, c'est Mike, ça lui va je crois mais clairement si c'était moi, je refuserai juste d'être CPT sur des projets Publik ou alors faut dispatcher. Surtout ça demanderait des allers/retours CPF/CPT/personne qui fait l'intégration qui ralentirait un projet express comme le CD06.

En attendant je retourne sur #28279 pour mesurer le boulot que ça représente vraiment.

Also available in: Atom PDF