Projet

Général

Profil

Bug #16476

dashboard : ne pas inclure les tuiles dans l'export de site

Ajouté par Frédéric Péters il y a presque 7 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
24 mai 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour le moment elles s'y retrouvent, genre :

      {
        "fields": {
          "extra_css_class": "",
          "restricted_to_unlogged": false,
          "key": "velov",
          "parameters": {
            "station": "Charpennes" 
          },.
          "last_update_timestamp": "2017-05-15T14:20:30.167Z",
          "order": 1,
          "public": true,
          "groups": [],
          "placeholder": "_dashboard",
          "slug": "" 
        },
        "model": "data.configjsoncell" 
      },

et c'est du bruit inutile. (vu le placeholder à _dashboard elles ne seront jamais affichées).

Je suggérerais de simplement zapper de l'export tout ce qui a placeholder.startswith('_').

Fichiers

0001-misc-don-t-export-cells-set-on-private-placeholders-.patch (968 octets) 0001-misc-don-t-export-cells-set-on-private-placeholders-.patch Frédéric Péters, 12 octobre 2017 11:19
0001-misc-don-t-export-cells-set-on-private-placeholders-.patch (3,71 ko) 0001-misc-don-t-export-cells-set-on-private-placeholders-.patch Frédéric Péters, 13 octobre 2017 10:44

Révisions associées

Révision 30ed33ea (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

misc: don't export cells set on private placeholders (#16476)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 6 ans

Voilà qui fait exactement ça.

#2

Mis à jour par Thomas Noël il y a plus de 6 ans

Ça supprime en même temps les cellules sans placeholder, ça me parait valable. Ack.

#3

Mis à jour par Frédéric Péters il y a plus de 6 ans

Ça supprime en même temps les cellules sans placeholder, ça me parait valable.

Sauf qu'alors les patchs devraient être mis à jour :/ Je m'en occupe.

#4

Mis à jour par Frédéric Péters il y a plus de 6 ans

Ajout de placeholder aux cellules créées dans les tests, pour qu'elles se retrouvent bien dans les exports.

#5

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)

Voilà avec les tests mis à jour.

commit 30ed33eaf457285cf04ef8a66e22535b0d40edd0
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Oct 12 11:18:13 2017 +0200

    misc: don't export cells set on private placeholders (#16476)
#6

Mis à jour par Frédéric Péters il y a plus de 5 ans

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

Formats disponibles : Atom PDF