bab85
Décembre 1, 2023, 11:55
21
J’ai laissé l’information de la base telle que dans le tuto (SQLITE_FILE_PATH=/var/lib/gladysassistant/gladys-production.db ), donc base de prod me semble-t-il. Par contre il a quand même fallu que je ré-appaire les capteurs. Je sais pas si c’est normal (c’est vrai qu’utilisant la base de prod, je m’attendais à avoir déjà mes capteurs reconnus). Dans z2m il n’y avait plus rien.
Le fait que tu me poses la questions je viens de faire une vérif. Je me rends compte que j’ai du faire une boulette , car je pensais que j’avais deux base z2m et non, quand je reviens sur mon gladys de prod, j’ai le même z2m avec seulement les 2 capteurs que je viens de tester, et j’ai une croix rouge entre Gladys et MQTT dans la configuration.
Je vais attendre de finir les tests pour ton développement, et je me remettrais propre ensuite.
Quel nom as tu donné pour le dossier qui contient la base et config?
Peux-tu aller dans la page de zigbee2mqtt et faire une capture d’écran de l’onglet Etat pour que je puisse voir tous les exposes de ton capteur?
Ci dessous un exemple.
bab85
Décembre 1, 2023, 12:43
23
Pour le nom du dossier j’ai du mettre /var/lib/gladysassistant_OWON_test: /var/lib/gladysassistant\
Et pour l’état voici :
Ok si tu as bien changé le nom alors ce n’est pas la même config que ta prod alors c’est une nouvelle instance.
Pour l’expose tamper il est bien présent, je regarde ce soir et je te tiens au courant.
1 « J'aime »
@bab85 ,
j’ai trouvé pourquoi tu ne voyais pas la fonction « tamper » sur ton capteur, un oubli de ma part
Je re-test de mon côté et je vais générer une nouvelle image.
Je te dit quand elle sera dispo.
Je te donnerai en même temps les commande pour mettre à jour l’image de ton côté.
@pierre-gilles , j’ai fais une bêtise de mon côté en voulant rebaser ma branche du coup la PR à été fermé.
Du coup la nouvelle PR
GladysAssistant:master
← William-De71:zigbee2mqtt-add-missing-exposes-OWON-PIR313-E
opened 10:25PM - 01 Dec 23 UTC
### 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)
- [ ] 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 ([french forum](https://community.gladysassistant.com/)/[english forum](https://en-community.gladysassistant.com/)) for testing before merging.
- [ ] 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.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
Add tamper and battery low expose in zigbee2mqtt integration
![image](https://github.com/GladysAssistant/Gladys/assets/11477113/73d36efd-6a72-4dcb-aa47-7a8d3eb53337)
L’image est prête
willde71/gladys-test:zigbee2mqtt-owon-pir313-e
Voici les commandes si tu veux re-tester
docker stop gladys
docker rm gladys
docker pull willde71/gladys-test:zigbee2mqtt-owon-pir313-e
docker run .....
pour la commande stop
et rm
mets bien le nom que tu avais utilisé lors de ton test.
Ensuite refait la commande docker run...
bab85
Décembre 2, 2023, 8:11
27
Bonjour,
J’ai testé la nouvelle image. Impeccable, tout fonctionne correctement. L’item détection de sabotage est disponible. Lorsque le détecteur est sur son support, on a la valeur « non » en vert, sinon « oui » en rouge. Pour la batterie, cela fonctionne toujours.
2 « J'aime »
@bab85 , merci pour ton test.
bab85
Décembre 3, 2023, 6:24
29
Merci surtout pour le développement et la rapidité de prise en compte.
Merci pour la PR @_Will_71 !
Je t’ai fais un retour :
GladysAssistant:master
← William-De71:zigbee2mqtt-add-missing-exposes-OWON-PIR313-E
opened 10:25PM - 01 Dec 23 UTC
### 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)
- [ ] 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 ([french forum](https://community.gladysassistant.com/)/[english forum](https://en-community.gladysassistant.com/)) for testing before merging.
- [ ] 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.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
Add tamper and battery low expose in zigbee2mqtt integration
![image](https://github.com/GladysAssistant/Gladys/assets/11477113/73d36efd-6a72-4dcb-aa47-7a8d3eb53337)
1 « J'aime »
J’ai vu je fais la modif ce soir.
1 « J'aime »
@pierre-gilles , j’ai fait la modification
Merci pour les correctifs @_Will_71 ! C’est testé et c’est bon pour moi
J’ai mergé et ça partira dans la prochaine version de Gladys !
4 « J'aime »