Projet

Général

Profil

Bug #41226

ne pas inclure les cellules invalides dans le tableau de bord

Ajouté par Frédéric Péters il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
01 avril 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

cf #41223; techniquement le is_visible() va être faux, ce qui va faire une 403, ce qui va faire que la cellule va s'afficher avec "undefined" comme erreur.


Fichiers

Révisions associées

Révision 9786d9b3 (diff)
Ajouté par Frédéric Péters il y a presque 4 ans

dashboard: skip invalid cells in dashboards (#41226)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 4 ans

#3

Mis à jour par Thomas Noël il y a presque 4 ans

  • Statut changé de Solution proposée à Solution validée

J'imagine que la technique que tu as choisie ici permet de limiter le nombre de requêtes SQL, mais avais-tu pensé à l'ajout d'une property "is_valid" sur Tile, qui irait chercher le ValidationInfo de chaque et en vérifierait le invalid_since ? Dans l'idée au final d'avoir, dans DashboardCell.render, un lisible tiles = [t for t in tiles if tiles.id_valid()] ?

Mais ce code me va bien aussi, de toute façon avec les Tile faut jouer des coudes :)

#4

Mis à jour par Frédéric Péters il y a presque 4 ans

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

Oui, c'est fait ainsi pour limiter le nombre de requêtes.

commit 9786d9b387df45e38df82f1794539aa6d160ddb0
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Apr 1 13:48:24 2020 +0200

    dashboard: skip invalid cells in dashboards (#41226)
#5

Mis à jour par Frédéric Péters il y a presque 4 ans

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

Formats disponibles : Atom PDF