Project

General

Profile

Développement #49173

dataviz: mettre le catalogue des visualisations dispo en db

Added by Valentin Deniaud almost 4 years ago. Updated almost 4 years ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
07 December 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

Related to Combo - Développement #48865: dataviz: récupérer les statistiques depuis d'autres briquesFermé26 November 2020

Actions

Associated revisions

Revision de584c39 (diff)
Added by Valentin Deniaud almost 4 years ago

tests: decrease indentation level in dataviz (#49173)

Revision 5db0931f (diff)
Added by Valentin Deniaud almost 4 years ago

tests: format dataviz using black (#49173)

Revision 356bb767 (diff)
Added by Valentin Deniaud almost 4 years ago

dataviz: save available visualizations in db (#49173)

History

#1

Updated by Valentin Deniaud almost 4 years ago

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.
#2

Updated by Valentin Deniaud almost 4 years ago

#3

Updated by Frédéric Péters almost 4 years ago

  • Status changed from Solution proposée to Solution validée

Ok.

#4

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)
#5

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.

#6

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é.

#7

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)
#8

Updated by Frédéric Péters almost 4 years ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF