Bonjour @Loic,
Sur le plugin Domogeek, il y a depuis longtemps un bug d’image.
Avant de faire mon premier plugin, je savais pas quoi corriger, maintenant je sais…
Je viens de te faire une PR : https://github.com/jeedom/plugin-domogeek/pull/14
J’en ai profité pour faire des changements léger sur l’affichage de l’Eqlogic. Comme ca on est plus proche du format actuel des plugins.
A l’occasion, si ca te va, accepte la PR et passe le en stable 
Super merci me servant pas du plugin j’ai merge on verra les retour
Merci
Je vais l’indiquer sur le forum comme ça y’aura des testeurs pour valider en stable.
Moi c’est la version sur laquelle je tourne 
Bonjour @Loic,
As tu prévu de prochainement basculer les modifications de l’été dernier en stable ?
Merci d’avance
J’ai pas vraiment eu de retour donc je sais pas si c’est bon ou pas…
Je comprend et pourtant il est beaucoup utilisé. Le souci étant que le plugin évolue peu donc les gens tourne surtout en stable.
D’autant plus que plus de support si le plugin est en beta.
Tu as pas en visu le nombre de personne qui utilise la beta ? Car si y’a du monde, depuis 6 mois ils se serait manifesté non ?
Je profite de ce thread pour te demander un truc. Je voudrais te faire un PR sur Domogeek pour avoir une commande de refresh. En effet, encore ce matin, mes volets se sont levé (ils auraient pas dû) car Domogeek s’était lancé à 00h et 4h mais mon déclencheur était à 3h (update crepuscule).
Pour éviter cela, j’aimerais ajouter dans mon scénario un refresh domogeek au début de mon scénario volet.
Mais avant de le dev, je préfère te demander au cas où tu voudrait pas l’accepter et donc l’avoir fait pour rien.
Vas y pour le pr et une fois fait rappel le moi je n’ai malheureusement pas assez de temps pour m’occuper de ce plugin
Ok.
Je te propose meme de le remettre au gout du jour.
Car je vois qu’il a était fait il y a très longtemps et le code n’est plus sur les standard de plugin
T’embête pas trop on prépare depuis plusieurs moi un nouveau truc donc ça sert a rien de passer du temps la dessus
ok. C’est bon a savoir avant d’y passer du temps.
Du coup tu pourras prévoir dans ce nouveau truc la capacité de forcer un refresh ?
C’est déjà le cas
au top alors 
Je viens de pousser une PR.
Je revu un peu le code d’affichage pour être plus proche des versions actuelles et ajouter la commande Refresh.
Je suis pas aller plus loin vu qu’une autre chose arrive.
J’ai fait une série de test et tout semble bon.
Merci pour le PR je viens de merger et de faire une stable
Oui c’est parfait merci.
Je viens de le mettre a jour sur ma prod. Tout semble fonctionner.
J’aurais du te mettre a jour le changelog. Ca va encore foudroyer sur Community 
Oui… Mais j’assumerais c’est ma faute le changelog puis j’ai l’habitude depuis le temps
En tout cas merci.
Ca me fera patienter le temps de la nouvelle version que vous préparez.
Pour info, j’ai l’impression que la PR 13 et 11 sont déjà corrigé dans ma version. Donc tu devrais pouvoir les clore.
Ce sujet a été automatiquement fermé après 24 heures suivant le dernier commentaire. Aucune réponse n’est permise dorénavant.
