Development #34235
import csv, état "finished !failed before finishing!"
Début:
22 juin 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Me laissant perplexe.
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Fichier 0001-manager-show-failure-only-in-the-running-sate-34235.patch 0001-manager-show-failure-only-in-the-running-sate-34235.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Fichier 0002-manager-update-translation-34235.patch 0002-manager-update-translation-34235.patch ajouté
- Fichier 0001-manager-show-failure-only-in-the-running-sate-34235.patch 0001-manager-show-failure-only-in-the-running-sate-34235.patch ajouté
C'est mieux avec une traduction.
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Perplexe aussi tu le fait qu'il y ait à faire dans le template un if/else pour connaître le réel état du machin, je m'attendrais à ce que ce genre de choses soit géré dans une couche plus basse.
Mais pour ne pas partir dans trop loin sur ce ticket qui ne parlait que d'affichage, on va dire ok.
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
Si tu valides le #34234 avant ça simplifiera les choses, il introduit une méthode pour avoir un état "traduit".
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
Ma remarque était sur le fait d'avoir à gérer dans le template la situation state == 'running' and not report.is_running
, je m'attendais à voir plus bas une méthode genre get_state
qui fait ce genre de vérif et qui renvoie un état 'consolidé'.
Bref vas-y comme ça.
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit e1838eb73117d652d436ec10289e67c7e08dbee2 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Sat Jun 22 22:27:10 2019 +0200 manager: add accessor to report state (#34235)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manager: add accessor to report state (#34235)