MMM-VigilanceMeteoFrance

[grena]

@BigFer said in MMM-VigilanceMeteoFrance:

Module mis à jour sans problème. Super pratique !

Super ! Merci ;)

Pour ton autre question, peux-tu ouvrir un sujet spécifique pour ne pas créer de hors-sujet dans celui-ci ? J’y répondrai avec plaisir :slightly_smiling_face:

Pour plus de lisibilité, n’hésites pas à utiliser le bouton " Code " </> pour coller tes lignes de code et ta configuration !

[omoioli]

Bonjour a tous, super module pour compléter la météo !
Il serait possible de pouvoir modifier le département à la volée par notification (en utilisation avec un module de voix par exemple …) pour avoir un aperçu de la vigilance dans plusieurs secteurs ?

[grena]

Bonjour @omoioli,

Oui c’est tout à fait possible grâce au système de notifications du Magic Mirror. Je n’utilise pas de système de contrôle vocal mais l’idée peu être étendue à l’utilisation de menus, de boutons, d’un système de rotation automatique… Et ça me semble intéressant :thumbsup:

Bref, j’ai intégré la fonctionnalité (fonctionnelle d’après mes tests) dans la branche testing ! Si tu es partant pour tester la fonctionnalité avec ton module de contrôle vocal, on peut en discuter en privé ;)

[omoioli]

Désolé, je viens juste de voir ton message, oui, ça m’intéresse de tester ça, je vois pour une mise à jour depuis ta branche testing.
Je suis pas mal occupé en ce moment, mais je te fais un retour dès que je peux.

[omoioli]

Je teste depuis quelques jours le module, mais je ne sais pas si cela marche, tout ce que je teste est en vigilance verte, c’est vrai qu’il fait beau un peu partout …
Il faudrait peut-être prévoir d’indiquer le numero de département à côté de la vigilance pour voir que le module s’est mis à jour …

[omoioli]

En fait si ça marche! J’avais mal orthographié la notification… maintenant le changement fonctionne bien.
Mais je pense que mettre le numéro de département au moins dans le header, ça pourrait être pas mal…
En tout cas merci pour ce module !

[grena]

@omoioli

En fait si ça marche! J’avais mal orthographié la notification… maintenant le changement fonctionne bien.

Super !

Il faudrait peut-être prévoir d’indiquer le numero de département à côté de la vigilance pour voir que le module s’est mis à jour …

En effet, c’est une bonne idée ;) Je viens tout juste d’intégrer la fonctionnalité dans la branche testing ! L’affichage du nom et du numéro du département n’est pas activé par défaut, il faut donc rajouter la ligne suivanteshowDepartment: true, dans ton fichier de configuration.

En tout cas merci pour ce module !

Avec plaisir :D J’attends ton retour final pour valider ces nouvelles fonctionnalités et les basculer dans la branche principale pour que tout le monde puisse en profiter !

[omoioli]

Salut,
Pour moi, tout a l’air de fonctionner, avec le département dans le header, ça change bien a chaque notification … Impeccable !
J’ai juste une petite remarque, qui n’est qu’un détail, mais j’ai regardé ton code du module pour voir les infos retournées par l’api, et j’ai remarqué que pour le header, tu indiquais le numéro de département demandé à la notification, mais tu pourrais peut-être indiquer le numéro de département retourné par l’api ?
Ca ne changerait rien au résultat final, c’est juste un changement de variable, mais ça permettrais d’être sur que le module a bien eu la réponse de l’api…
Je sais pas si je suis clair … mais comme je t’ai dit c’est juste un détail…
En tout cas merci pour ton module !

[grena]

@omoioli said in MMM-VigilanceMeteoFrance:

Ca ne changerait rien au résultat final, c’est juste un changement de variable, mais ça permettrais d’être sur que le module a bien eu la réponse de l’api…

En fait, si tu regardes la fonction processVigi, je teste (sommairement) la conformité des données reçues pour l’affichage final. Il y a notamment data.department != this.config.department pour détecter si le département n’est pas le bon…

En cas de données non utilisables, le module reste bloqué sur " Chargement… ", on voit donc très rapidement qu’il y a un problème ;)

[omoioli]

Autant pour moi, j’ai regardé vite fait le code, je l’ai pas assez détaillé…
Donc je n’ai plus rien à dire, le module est parfait !
Merci !!!