Développement #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.
Files
Related issues
Associated revisions
tests: format dataviz using black (#49173)
dataviz: save available visualizations in db (#49173)
History
Updated by Valentin Deniaud almost 4 years ago
- File 0001-tests-decrease-indentation-level-in-dataviz-48865.patch 0001-tests-decrease-indentation-level-in-dataviz-48865.patch added
- File 0003-dataviz-save-available-visualizations-in-db-48865.patch 0003-dataviz-save-available-visualizations-in-db-48865.patch added
- File 0002-tests-format-dataviz-using-black-48865.patch 0002-tests-format-dataviz-using-black-48865.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
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.
Updated by Valentin Deniaud almost 4 years ago
- Related to Développement #48865: dataviz: récupérer les statistiques depuis d'autres briques added
Updated by Frédéric Péters almost 4 years ago
- Status changed from Solution proposée to Solution validée
Ok.
Updated by Frédéric Péters almost 4 years ago
- Status changed from Solution validée to 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)
Updated by Valentin Deniaud almost 4 years ago
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.
Updated by Frédéric Péters almost 4 years ago
- Status changed from Solution proposée to Solution validée
Ok je vais créer un nouveau ticket sur le sujet; celui-ci validé.
Updated by Valentin Deniaud almost 4 years ago
- Status changed from Solution validée to 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)
Updated by Frédéric Péters almost 4 years ago
- Status changed from Résolu (à déployer) to Solution déployée
tests: decrease indentation level in dataviz (#49173)