Development #49173
dataviz: mettre le catalogue des visualisations dispo en db
0%
Description
La finalité c'est de pouvoir aller chercher ces visualisations sur plusieurs site, dans ce cas il ne sera pas possible de continuer à fonctionner comme actuellement, en faisant une requête à chaque affichage du formulaire plus une autre à chaque sauvegarde.
À la place donc, s'orienter vers une récupération de ce qui est dispo toutes les heures.
Fichiers
Demandes liées
Révisions associées
tests: format dataviz using black (#49173)
dataviz: save available visualizations in db (#49173)
Historique
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0001-tests-decrease-indentation-level-in-dataviz-48865.patch 0001-tests-decrease-indentation-level-in-dataviz-48865.patch ajouté
- Fichier 0003-dataviz-save-available-visualizations-in-db-48865.patch 0003-dataviz-save-available-visualizations-in-db-48865.patch ajouté
- Fichier 0002-tests-format-dataviz-using-black-48865.patch 0002-tests-format-dataviz-using-black-48865.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Comme c'est le début d'un gros travail sur cette cellule, je commence par deux patches triviaux pour améliorer les tests.
Ensuite, 0003 pour traiter le ticket. Points à noter :- Data migration pour mettre à jour les cellules existantes. Elle est splittée en 3 fichiers parce que postgres ne tolère pas qu'on change les données et le schéma en même temps (c'est la théorie, en local pour moi ça passait avec migrate_schemas, mais plus au moment d'écrire un test).
- Il y avait du code pour préfixer le nom des visus avec le nom du site au cas où il y aurait eu plusieurs bijoe. En pratique je suppose que ce n'est le cas nulle part. Mais ici on part d'ores et déjà du principe qu'il y aura plusieurs sites, et donc on affiche ce préfixe systématiquement (il passe donc de « jamais affiché » à « toujours affiché »). Problème, bijoe s'appelle par défaut « Statistiques », j'ai peur que le rendu ne soit pas top et je suis preneur d'idées pour faire mieux.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Lié à Development #48865: dataviz: récupérer les statistiques depuis d'autres briques ajouté
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Ok.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution validée à Solution proposée
Ah non j'oubliais je m'étais noté cette erreur dans un coin :
Traceback (most recent call last):
File "/home/fred/src/eo/combo/combo/data/management/commands/cron.py", line 45, in handle
appconfig.hourly()
File "/home/fred/src/eo/combo/combo/apps/dataviz/__init__.py", line 35, in hourly
self.update_available_statistics()
File "/home/fred/src/eo/combo/combo/apps/dataviz/__init__.py", line 55, in update_available_statistics
result = requests.get(
File "/usr/lib/python3/dist-packages/requests/models.py", line 900, in json
return complexjson.loads(self.text, **kwargs)
File "/usr/lib/python3.9/json/__init__.py", line 346, in loads
return _default_decoder.decode(s)
File "/usr/lib/python3.9/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib/python3.9/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
Mis à jour par Valentin Deniaud il y a plus de 3 ans
Frédéric Péters a écrit :
Ah non j'oubliais je m'étais noté cette erreur dans un coin :
Donc ajouter un try/except pour ne pas crasher si on ne récupère pas du JSON ? OK mais ce .json() violent c'est rien que du code déplacé de ce qui était déjà là, je préférerais un autre ticket pour m'éviter les rebases relous.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Ok je vais créer un nouveau ticket sur le sujet; celui-ci validé.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 356bb767f511a53f9ff580477f5b0f05a79495ee
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Thu Nov 26 16:09:26 2020 +0100
dataviz: save available visualizations in db (#49173)
commit 5db0931f104ed6049f72b941e378180837df80a6
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Thu Nov 26 16:32:09 2020 +0100
tests: format dataviz using black (#49173)
commit de584c394f739e121b2501f9f3af5078c4e8d5ee
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Thu Nov 26 16:21:42 2020 +0100
tests: decrease indentation level in dataviz (#49173)Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
tests: decrease indentation level in dataviz (#49173)