Projet

Général

Profil

Development #68140

Cellules fiche: merger la cellule fiches et la cellule contenu d'une fiche

Ajouté par Lauréline Guérin il y a plus d'un an. Mis à jour il y a plus d'un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
12 août 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Suite de #68015 #68037 #68063

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

0004-wcs-add-a-css-class-depending-on-display-mode-on-car.patch (941 octets) 0004-wcs-add-a-css-class-depending-on-display-mode-on-car.patch Lauréline Guérin, 12 août 2022 15:37
0005-wcs-snapshot-loading-with-old-card-cells-68140.patch (9,66 ko) 0005-wcs-snapshot-loading-with-old-card-cells-68140.patch Lauréline Guérin, 12 août 2022 15:37
0002-wcs-migration-for-card-cell-merge-68140.patch (4,1 ko) 0002-wcs-migration-for-card-cell-merge-68140.patch Lauréline Guérin, 12 août 2022 15:37
0003-wcs-delete-old-card-cell-models-68140.patch (58,4 ko) 0003-wcs-delete-old-card-cell-models-68140.patch Lauréline Guérin, 12 août 2022 15:37
0001-wcs-add-new-card-cell-model-68140.patch (9,81 ko) 0001-wcs-add-new-card-cell-model-68140.patch Lauréline Guérin, 12 août 2022 15:37
0001-scss-compatibility-with-new-card-cell-68140.patch (1,93 ko) 0001-scss-compatibility-with-new-card-cell-68140.patch Lauréline Guérin, 12 août 2022 15:37
0005-wcs-snapshot-loading-with-old-card-cells-68140.patch (9,66 ko) 0005-wcs-snapshot-loading-with-old-card-cells-68140.patch Lauréline Guérin, 12 août 2022 15:48
0005-wcs-snapshot-loading-with-old-card-cells-68140.patch (9,66 ko) 0005-wcs-snapshot-loading-with-old-card-cells-68140.patch Lauréline Guérin, 29 août 2022 14:39
0004-wcs-add-a-css-class-depending-on-display-mode-on-car.patch (941 octets) 0004-wcs-add-a-css-class-depending-on-display-mode-on-car.patch Lauréline Guérin, 29 août 2022 14:39
0002-wcs-migration-for-card-cell-merge-68140.patch (4,1 ko) 0002-wcs-migration-for-card-cell-merge-68140.patch Lauréline Guérin, 29 août 2022 14:39
0003-wcs-delete-old-card-cell-models-68140.patch (58,4 ko) 0003-wcs-delete-old-card-cell-models-68140.patch Lauréline Guérin, 29 août 2022 14:39
0001-wcs-add-new-card-cell-model-68140.patch (10,4 ko) 0001-wcs-add-new-card-cell-model-68140.patch Lauréline Guérin, 29 août 2022 14:39

Demandes liées

Lié à Combo - Development #68531: Cellule fiche(s) : Liste "Fiche(s) à afficher"Fermé30 août 2022

Actions

Révisions associées

Révision 797f9af2 (diff)
Ajouté par Lauréline Guérin il y a plus d'un an

wcs: add new card cell model (#68140)

Révision d8fcc66f (diff)
Ajouté par Lauréline Guérin il y a plus d'un an

wcs: migration for card cell merge (#68140)

Révision 3a3eae69 (diff)
Ajouté par Lauréline Guérin il y a plus d'un an

wcs: delete old card cell models (#68140)

Révision 4019081d (diff)
Ajouté par Lauréline Guérin il y a plus d'un an

wcs: add a css class depending on display mode on card cell (#68140)

Révision 574228c0 (diff)
Ajouté par Lauréline Guérin il y a plus d'un an

wcs: snapshot loading with old card cells (#68140)

Historique

#1

Mis à jour par Lauréline Guérin il y a plus d'un an

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

#3

Mis à jour par Frédéric Péters il y a plus d'un an

(j'ai poussé le patch côté publik-base-theme)

#4

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

#5

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

#6

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

#8

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.

#9

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)
#10

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 :
Ce serait mieux d'avoir :
  • Toutes les fiches
  • Certaines fiches
#11

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

#12

Mis à jour par Transition automatique il y a plus d'un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#13

Mis à jour par Pierre Cros il y a plus d'un an

#14

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.

D'autre part, en l'état, dans le mode d'affichage Carte :
  • Disposition de la grille : Automatique / Modifier (Le bouton modifier est inopérant)
  • Idem pour bouton "Ajouter"
#15

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)

#16

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

#17

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF