C’est vrai ça, bon anniversaire @cicoub13 !!
VonOx
April 13, 2022, 6:58pm
22
Bon anniversaire @cicoub13
Heureusement qu’il y a Alex, bon anniversaire @cicoub13 .
1 Like
Bon anniversaire @cicoub13 !!
Et bravo pour la vidéo !! Tres belle installation !!
Bon anniversaire @cicoub13
Merci à vous deux pour cette super vidéo claire et bien documentée, elle apporte encore de nouveaux horizons, bravo !
@cicoub13 , j’ai essayé de reproduire ta scène pour l’envoie de la météo avec la requête HTTP sur l’api de OpenWeather. C’est cool j’ai appris que Gladys avait la possibilité de créer des variables automatiquement sur les retour JSON.
Mais est ce que tu as des problèmes quand tu veux lire les variables de température mini et maxi?
Car après avoir sauvegardé la scène, les _min et _max de chaque variables sont retirés
Pourtant ces variables apparaissent bien dans la liste
Donc quand je reçoit le message, du coup je reçoit à la place la variable data.main.temp qui ne correspond pas au min et max
bjm
April 14, 2022, 11:50am
29
Je n’ai pas encore vue la vidéo mais
Bon anniversaire @cicoub13
Hehe, il y a effectivement un bug et je l’avais corrigé pour le moment à la main dans mon instance (d’ailleurs la mise à jour de la dernière version a remplacé mon code et donc réintroduit le bug, comme tu peux le voir dans la vidéo).
J’ai créé l’issue [Scene] Bad variable management when one variable id contains another id variable · Issue #1503 · GladysAssistant/Gladys · GitHub et fait une proposition de correction fix: Sort variables to avoid bad replacement in text by cicoub13 · Pull Request #1504 · GladysAssistant/Gladys · GitHub
Je dois jeter un coup d’oeil pour améliorer les tests
3 Likes
Merci à tous pour vos messages d’anniversaire
2 Likes
cicoub13:
Pour répondre à une des questions de la vidéo, les détecteurs de mouvement envoient un "occupancy": false,
après 90 secondes (paramètre par défaut, configurable dans Zigbee2Mqtt).
Sur les Xiaomi c’est configurable mais sur les capteurs de mouvement sonoff SONOFF SNZB-03 control via MQTT | Zigbee2MQTT , ce paramètre occupancy
n’est pas configurable.
Comment on peux gérer ça ?
VonOx
April 28, 2022, 10:53am
33
Bah on peut pas, c’est pas réglable.
J’ai un sonoff et ça doit être 2 minutes la tempo
Edit : c’est 1 minute
Ah, je m’en doutais un peux.
Hello @cicoub13 @pierre-gilles ,
Est-ce qu’il serait possible d’avoir de la visibilité sur la livraison de cette correction ?
Je viens de proposer une nouvelle correction. En attente de relecture
GladysAssistant:master
← cicoub13:fix-1503
opened 05:43AM - 15 Apr 22 UTC
Fix #1503
### Pull Request check-list
To ensure your Pull Request can be a… ccepted as fast as possible, make sure to review and check all of these items:
- [x] If your changes affects code, did your write the tests?
- [x] Are tests passing? (`npm test` on both front/server)
- [x] Is the linter passing? (`npm run eslint` on both front/server)
- [x] Did you run prettier? (`npm run prettier` on both front/server)
- [x] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [x] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to [the community](https://community.gladysassistant.com/) for testing before merging.
- [x] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
- [x] If you are adding a new features/services which needs explanation, did you modify the user documentation? See [the GitHub repo](https://github.com/GladysAssistant/v4-website) and the [website](https://gladysassistant.com).
- [x] Did you add fake requests data for the demo mode (`front/src/config/demo.js`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](https://demo.gladysassistant.com).
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
Please provide a description of the change here. It's always best with screenshots, so don't hesitate to add some!
1 Like
Merci pour l’info. Est-ce qu’il y a besoin d’aide pour des tests ?
1 Like
Salut @cicoub13 , je viens de tester ta PR et c’est bon ça corrige bien le bug de mon côté J’ai mergé, ça partira dans la prochaine version de Gladys !
3 Likes