Gladys Assistant 4.2 est disponible avec la compatibilité Zigbee2mqtt ! 🚀

pierre-gilles · Mai 7, 2021, 2:09

@AlexTrovato j’ai fais une PR mais j’ai un test sur le bluetooth que j’ai pas réussi à corriger:

github.com/GladysAssistant/Gladys

Check if bluetooth is ready before startScanning

GladysAssistant:master ← GladysAssistant:check-if-bluetooth-is-ready

opened 02:07AM - 07 May 21 UTC

Pierre-Gilles

+4 -1

### Pull Request check-list To ensure your Pull Request can be accepted as fa…st as possible, make sure to review and check all of these items: - [ ] If your changes affects code, did your write the tests? - [ ] Are tests passing? (`npm test` on both front/server) - [ ] Is the linter passing? (`npm run eslint` on both front/server) - [ ] Did you run prettier? (`npm run prettier` on both front/server) - [ ] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front) - [ ] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code) - [ ] 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). - [ ] Did you add fake requests data for the demo mode (`front/src/config/demo.json`) 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!

Le test en question:

Si jamais tu passe dans le coin

J’essaie de comprendre le test mais j’ai du mal à comprendre ce qui est testé ici, je voudrais éviter de modifier ça à l’aveugle en perdant le sens du test

AlexTrovato · Mai 7, 2021, 10:51

Désolé pas beaucoup de temps en ce moment. Je regarde asap.

pierre-gilles · Mai 7, 2021, 11:13

T’inquiète pas, je vais continuer à investiguer de mon côté

pierre-gilles · Mai 13, 2021, 2:06

Pour information, j’ai continué à investiguer de mon côté ce matin avant la release d’aujourd’hui, mais rien n’à faire je n’arrive pas à fixer les tests du service Bluetooth, et je pense que c’est le signe que je ne comprend pas l’intégration

Je ne vais donc pas pousser mes modifications qui feront sans doute plus de dégâts que de correctifs, et il faudra attendre le retour de @AlexTrovato pour avoir un fix sur ce bug bluetooth.

spenceur · Mai 13, 2021, 8:17

Mon Gladys est up et depuis le service fonctionne très bien.
Du coup j’en déduis qu’il suffit de restart le container pour corrigé temporairement le service bluetooth !
Si ça peut aider d’autre utilisateur !
Gg pour la release @pierre-gilles

pierre-gilles · Mai 13, 2021, 8:39

C’est vraiment aléatoire, c’est pour ça qu’il est assez compliqué à reproduire ce bug.

Il faut que le service bluetooth ne soit pas ready au moment ou Gladys démarre.

Mastho · Mai 17, 2021, 5:19

Je viens d’activer l’intégration Zigbee2Mqtt sur ma nouvelle installation et tout a démarré correctement du premier coup. Le Bluetooth fonctionne toujours donc le bug dont je parlais plus haut ne s’est pas reproduit