Development #68140
Cellules fiche: merger la cellule fiches et la cellule contenu d'une fiche
0%
Description
La cellule contenu d'une fiche peut afficher plusieurs fiches, on a aussi maintenant un mode tableau qui reprend la mise en page de la cellule fiches.
Merger ces deux cellules pour n'en garder qu'une, qué s'appelerio "Fiche(s)".
Au niveau du model, j'aimerais renommer WcsCardInfosCell en WcsCardCell, pour simplifier.
Attention lors du merge et renommage aux points suivants:
- les assets, il faut qu'ils continuent à fonctionner avec les conf existantes
- les css: prévoir une transition pour que les classes css des anciennes cellules continuent à fonctionner, puis nettoyage
- les templates: vérifier qu'il n'y a pas de surcharge cliente (normalement ok)
- export/import: vérifier les impacts
- generic foreign keys ?
- ... ?
Fichiers
Demandes liées
Révisions associées
wcs: migration for card cell merge (#68140)
wcs: delete old card cell models (#68140)
wcs: add a css class depending on display mode on card cell (#68140)
wcs: snapshot loading with old card cells (#68140)
Historique
Mis à jour par Lauréline Guérin il y a plus d'un an
- Fichier 0005-wcs-snapshot-loading-with-old-card-cells-68140.patch 0005-wcs-snapshot-loading-with-old-card-cells-68140.patch ajouté
- Fichier 0004-wcs-add-a-css-class-depending-on-display-mode-on-car.patch 0004-wcs-add-a-css-class-depending-on-display-mode-on-car.patch ajouté
- Fichier 0003-wcs-delete-old-card-cell-models-68140.patch 0003-wcs-delete-old-card-cell-models-68140.patch ajouté
- Fichier 0002-wcs-migration-for-card-cell-merge-68140.patch 0002-wcs-migration-for-card-cell-merge-68140.patch ajouté
- Fichier 0001-wcs-add-new-card-cell-model-68140.patch 0001-wcs-add-new-card-cell-model-68140.patch ajouté
- Fichier 0001-scss-compatibility-with-new-card-cell-68140.patch 0001-scss-compatibility-with-new-card-cell-68140.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
0001: nouveau model WcsCardCell
0002: migration pour passer des anciennes cellules à la nouvelle (je ne gère pas le retour arrière)
0003: suppression des anciens models
0004: ajout d'une class css pour le mode tableau et le mode carte, pour aller avec 0001 dans publik-base-theme
0005: faire en sorte que l'export d'un snapshot avec des anciennes cellules fonctionne quand même
pour la migration: ça marche bien en local, et je suis sereine à 99%. Pour les 1% restants, on fait comment en cas de plantage ? :)
basé sur #68063
Mis à jour par Lauréline Guérin il y a plus d'un an
Mis à jour par Frédéric Péters il y a plus d'un an
Je serais pour en profiter ici pour renommer (dans 0001 "wcs: add new card cell model"),
+ related_card_path = models.CharField(_('Card Identifier'), max_length=1000, blank=True)
actuellement "Identifiant de fiche", pour avoir "Fiche(s) à afficher" ("Card(s) to display"); qui me semble plus clair avec le merge.
Je pense qu'on gagnerait aussi à modifier "Other Card Identifiers" mais là je me dis que ça serait peut-être différent et hors de ce ticket (je pense à renommer l'option "Autres identifiants de fiche" du select en simplement "Autres:" + ne pas afficher le libellé pour le champ "Other Card Identifiers", qui n'apparait que quand "Autres" est sélectionné).
Mis à jour par Pierre Cros il y a plus d'un an
"Fiche(s) à afficher" c'est bien mais on peut garder identifiant en remarque en dessous du champ ? Genre "Identifiant(s) séparés par des virgules" (je sais plus s'il faut les séparer par des virgules ou fournir une objet liste ou quoi, c'est juste pour illustrer).
Mis à jour par Frédéric Péters il y a plus d'un an
mais on peut garder identifiant en remarque en dessous du champ ?
Le champ en question c'est un <select> mais le champ dessous, "autres", où je propose de ne pas afficher le libellé, pourrait avoir l'info comme quoi ce sont des identifiants qui sont attendus, en remarque. (perso je gardais donc ça pour un ticket à créer derrière).
Mis à jour par Lauréline Guérin il y a plus d'un an
- Fichier 0005-wcs-snapshot-loading-with-old-card-cells-68140.patch 0005-wcs-snapshot-loading-with-old-card-cells-68140.patch ajouté
- Fichier 0004-wcs-add-a-css-class-depending-on-display-mode-on-car.patch 0004-wcs-add-a-css-class-depending-on-display-mode-on-car.patch ajouté
- Fichier 0003-wcs-delete-old-card-cell-models-68140.patch 0003-wcs-delete-old-card-cell-models-68140.patch ajouté
- Fichier 0002-wcs-migration-for-card-cell-merge-68140.patch 0002-wcs-migration-for-card-cell-merge-68140.patch ajouté
- Fichier 0001-wcs-add-new-card-cell-model-68140.patch 0001-wcs-add-new-card-cell-model-68140.patch ajouté
Rebase.
changement de verbose_name pour le champ related_card_path
(model & form) dans 0001
Mis à jour par Frédéric Péters il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Go ainsi.
Mis à jour par Lauréline Guérin il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 574228c0b8ead466a775880c95e8da27094ba7bb Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Aug 12 15:35:24 2022 +0200 wcs: snapshot loading with old card cells (#68140) commit 4019081d34fd188887fbf81608bdbd2c79efdf9f Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Aug 12 14:18:28 2022 +0200 wcs: add a css class depending on display mode on card cell (#68140) commit 3a3eae69ae680a2ef4f787f41e8ec03fba5c8032 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Aug 12 13:49:40 2022 +0200 wcs: delete old card cell models (#68140) commit d8fcc66f4b7123ed2721cbf859fc5b49b6e1a2a2 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Aug 12 11:40:36 2022 +0200 wcs: migration for card cell merge (#68140) commit 797f9af235e66fd71c6352e5af647f1809b7efce Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Aug 12 11:00:54 2022 +0200 wcs: add new card cell model (#68140)
Mis à jour par Pierre Cros il y a plus d'un an
C'est bien plus clair qu'avant déjà, vraiment cool.
Je le mets ici mais j'ouvre un autre ticket si vous préférez, "Fiche(s) à afficher" liste 2 éléments :- Toutes les fiches
- Autres :
- Toutes les fiches
- Certaines fiches
Mis à jour par Frédéric Péters il y a plus d'un an
un autre ticket
Oui plutôt un nouveau ticket; mais attention il peut y avoir davantage d'éléments dans cette liste (pour prendre en compte l'identifiant de fiche présent dans l'URL ou les relations entre fiches, et toutes ces autres situations sont en quelque sorte "certaines fiches").
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Pierre Cros il y a plus d'un an
- Lié à Development #68531: Cellule fiche(s) : Liste "Fiche(s) à afficher" ajouté
Mis à jour par Chloé Girard il y a plus d'un an
Personnellement je ne comprends pas l'usage du terme "Carte" pour l'affichage "Fiche". Je mettrais plutôt "Fiche", non ? Ou autre, à définir.
J'ai l'impression que "Carte" vient du "Cards" en anglais, mais en français ça évoque bien plus une carte géographique. On est deux, avec Brice, à avoir cet avis.
- Disposition de la grille : Automatique / Modifier (Le bouton modifier est inopérant)
- Idem pour bouton "Ajouter"
Mis à jour par Stéphane Laget il y a plus d'un an
cela pourrait être "tuile"
(mais on pourrait aussi penser que c'est confusant avec des couches géographiques)
Mis à jour par Frédéric Péters il y a plus d'un an
Attention ce ticket est déployé etc. pour une discussion effective il y a peut-être un meilleur endroit, et une fois une option retenue, un nouveau ticket combo.
(et pour info le terme consacré dsfr, https://www.systeme-de-design.gouv.fr/elements-d-interface/composants/carte (mais bien sûr on n'y est pas tenu, ils n'ont pas la confusion avec la carte géographique à gérer)).
wcs: add new card cell model (#68140)