search

overfl0w11 / MMM-Netatmo-Thermostat

Netatmo module for Magic Mirror
MIT License
3 stars 2 forks source link

fix bug in readme.md #4

Closed ColonelMoutarde closed 6 years ago

overfl0w11 commented 6 years ago

Salut @ColonelMoutarde . Au vu de ton pseudo, je suppose que tu parle Francais ^^

Merci pour ton PR. Cependant, je ne pense pas que ce soit nécessaire de passer l'update interval a 1 minute. En effet, dans la doc Netatmo, il est bien spécifié que le thermostat envoi ses données sur le serveur Netatmo toutes les heures. Il n'est donc pas nécessaire de demander toutes les minutes si les valeurs ont changées.

Je vais donc refuser ton PR, a moins que tu es un contre argument valable.

ColonelMoutarde commented 6 years ago

Effectivement, je suis français :) J'ai fait cette pr car la valeur par défaut affichée dans le tableau plus bas indique 1 minute (60 000). C'est pour mettre en cohérence.

PS: La couleur du fond ne change pas même si ma chaudière est en chauffe. Par contre la température affichée est bonne.

overfl0w11 commented 6 years ago

En effet, ce n'était pas cohérent avec le tableau. J'ai mis à jour ce dernier à l'instant.

Concernant ton problème de couleur qui ne change pas, je veux bien que tu créer une issues pour tracer ca. Je regarderais ca ce soir si j'ai un peu de temps. Je clos ce PR.