PnCevennes / app_oeasc

GNU General Public License v3.0
0 stars 0 forks source link

Système d'alerte : restitution #35

Open BaptisteAlgoet opened 3 years ago

BaptisteAlgoet commented 3 years ago

Je me remets au boulot sur le site de l'OEASC. J'aimerais qu'on parvienne à boucler la page de restitution du système d'alerte (page grand public).

Je crée donc cet "issue" pour y poser mes questions, réflexions et choses à faire.

Q1 : quand j'insère dans la page "restitution" le code du widget copié à partir de la page "DEV" : est-ce que la carte ou le graphique créé se met à jour tout seul lorsqu'une nouvelle alerte est signalée ?

Car le problème si je fais des exports au format image, c'est que tant que je ne refais pas tout manuellement, ça ne se mettra pas à jour automatiquement...

joelclems commented 3 years ago

Il faut que je m'y remette aussi

le carte ou le graphique suivent bien l'évolution des données

BaptisteAlgoet commented 3 years ago

Bug 1 : Si je copie / colle plusieurs widgets de cartes / graphiques (exemple : 1 carte + 1 graphique), et que je modifie la hauteur du 2e (exemple : le graphique), ça me modifie la hauteur des 2 (la carte ET le graphique).

Q2 : est-ce qu'il est serait possible de prévoir de régler à la fois la hauteur et la largeur des widget copiés/collés ? (facultatif) Bonus : pouvoir mettre 2 widgets côte à côte...

BaptisteAlgoet commented 3 years ago

A1 ("A" pour "amélioration") : lorsque l'on copie / colle un widget, ce serait bien que la ligne "Données = XXX/XXX" n'apparaisse pas.

image

BaptisteAlgoet commented 3 years ago
joelclems commented 3 years ago


- suppression du compteur de données
- choix du secteur (par filtre) dans les résultats
joelclems commented 3 years ago

https://oeasc.cevennes-parcnational.net/front/#/content/declaration_test

BaptisteAlgoet commented 3 years ago

Super. Je fais un test aujourd'hui pour voir si j'arrive à organiser les choses comme je veux dans cette page test...

BaptisteAlgoet commented 3 years ago

A priori, le filtre interactif "secteur" ne fonctionne pas si je veux ajouter des widgets.

J'ai du ajouter manuellement "secteur":contentValues.secteur ? [contentValues.secteur] : [] dans la catégorie "Filters" du widget. Mais j'imagine qu'on ne peut pas faire autrement, car sinon, le contenu des autres widgets (ceux du haut de la page) sera également filtré, même s'il ne doit pas l'être ?

Il faudra en tout cas prévoir un "mode d'emploi" synthétique pour préciser comment gérer / modifier le contenu de ces pages.

joelclems commented 3 years ago

C'est exactement ça. Il faut que je fasse en sorte que seul les graphiques concernés par ce filtre soient redessinés

BaptisteAlgoet commented 3 years ago

Et c'est faisable automatiquement ? Ou ça ne pourra se faire que manuellement ? (si trop compliqué de l'automatiser, ce n'est pas grave... ça ne vaut pas la peine d'y passer trop de temps)

Mais tu peux déjà commencer à faire regrouper quelque part, dans une sorte de mémo / tuto / mode d'emploi, tout ce qu'il faut savoir pour pouvoir gérer correctement le contenu du site), qui listerait, par exemple :

Ça me semble indispensable pour que ceux qui prendront le relai puissent gérer le site, une fois que nous ne serons plus là...

BaptisteAlgoet commented 3 years ago
BaptisteAlgoet commented 3 years ago

Est-il possible d'actualiser un texte automatiquement, selon une donnée spécifiée. Par exemple : "A ce jour, XX (nombre d'alertes) alertes ont été signalées. YY (nombre de dégâts) y ont été signalés."

Si c'est faisable facilement, dis-moi comment. Si c'est trop compliqué, tant pis, on n'y passe pas trop de temps. C'est assez accessoire.

BaptisteAlgoet commented 3 years ago

OK, j'ai réussi à monter sans souci un contenu de page (ici : https://oeasc.cevennes-parcnational.net/front/#/content/declaration_test) A part les quelques petits points d'améliorations listés ci-dessus (et dans l'issue concernant la page "DEV"), c'est vraiment super. Il y a de quoi être autonome pour éditer toutes les restitutions que l'on veut.

On est donc quasiment tout bon sur ce point ! Du super boulot ! 🥇

joelclems commented 3 years ago
BaptisteAlgoet commented 3 years ago

Bizarre : la couleur pour le massif Aigoual est passée de rouge à gris (cf. ci-dessous). C'était mieux en rouge, car sur nos cartes, chaque massif a une couleur qui lui est attribuée (cf. https://oeasc.cevennes-parcnational.net/front/#/observatoire/perimetre), et pour l'Aigoual, c'est le rouge, donc j'aimais mieux comme c'était avant... Tu pourrais le repasser en rouge, stp ?

image

joelclems commented 3 years ago

C'est le fait de changer le nom des secteurs qui déclenchent ces bugs, car il sont en dur en peu partout. Ce sera corrigé sous peu, l'idéal étant qu'il ne soient plus en dur, ou qu'on puisse passer par les code qui eux varient moins

BaptisteAlgoet commented 3 years ago

OK. En effet, le mieux, à la limite, ce serait de passer par les codes (AIGO, MTLO, CAUS, VALC) qui normalement, sont figés. La difficulté, c'est qu'en terme d'affichage à l'écran, il faut conserver les noms entiers...

Bon, cela dit, de toute façon, maintenant, les noms des secteurs ne sont plus censés bougés non plus : Aigoual, Mont Lozère, Causses-Gorges et Vallées cévenoles. Il y avait juste le "Mont Aigoual" à passer en "Aigoual". Maintenant que c'est fait, ça ne bougera plus.

joelclems commented 3 years ago

Fait Ctrl MAJ R au besoin

BaptisteAlgoet commented 3 years ago

Un bug demeure, et j'imagine que c'est lié : sur la page de restitution, le bouton affiche à nouveau "Mont Aigoual" (et non "Aigoual"), et quand on clique sur ce bouton, les graphiques chargent, mais ne s'affichent jamais... (les autres secteurs fonctionnent correctement...)

image

joelclems commented 3 years ago

C''est (re)corrigé, est ce que tu n'a pas copié la page depuis une page de test qui contient ('Mont Aigoual' au lieu de aigoual)

BaptisteAlgoet commented 3 years ago

Bah non, je crois pas... J'ai pas touché à cette partie là depuis le copier/coller que j'avais fait quand on a fait le point ensemble, mardi... Enfin bon, bref, pas grave. Ça fonctionne sans souci, maintenant.

joelclems commented 3 years ago

nb dégat fait, présent au début des résultats, peut être à remettre en forme

joelclems commented 3 years ago

quelques changements (encours)

joelclems commented 3 years ago

Pas mal de changement, merci de tester si il reste quelque bug (surtout avec le copier coller)

BaptisteAlgoet commented 3 years ago

A priori tout fonctionne correctement ! 👍

joelclems commented 3 years ago

Cool, on peut fermer cette issue et laisser ouverte la #36 sur la restitution dev pour les pb de couleur et pour les graphes relatifs

BaptisteAlgoet commented 3 years ago

Bug :

Le contenu de la carte "gravité max" dans la page "restitution" n'est pas le même que sur celle de la page "DEV". Exemple avec un zoom sur les vallées cévenoles :

page "restitution" (grand public) :

image

page "DEV" : image

Et a priori, c'est la carte de la page DEV qui est la bonne. A corriger le plus rapidement possible (si c'est possible ce matin, c'est bien, car j'ai une réunion avec l'ONF et les chasseurs à 14h !) !

BaptisteAlgoet commented 3 years ago

Et autre bug à corriger : les fonds de carte Mapbox et IGN (cartes) des cartes interactives ne fonctionnent plus.

BaptisteAlgoet commented 3 years ago

PS : a priori, le problème ne concerne pas que les cartes, mais les graphiques aussi.

exemple : page grand public : image

page DEV : image

(Vallées cévenoles : 1 seule alerte "INFR" sur la page grand public, alors qu'il y en a 3, en réalité (cf. page DEV)

D'ailleurs, on le voit aussi quand on regarde le nombre d'alertes par type de dégât dans la légende.