bab85
1. Dezember 2023 um 11:55
21
Ich habe die Datenbankinformationen so wie im Tutorial belassen (SQLITE_FILE_PATH=/var/lib/gladysassistant/gladys-production.db ), also scheinbar die Produktionsdatenbank. Allerdings musste ich die Sensoren trotzdem neu paaren. Ich weiß nicht, ob das normal ist (es stimmt, dass ich, da ich die Produktionsdatenbank verwende, erwartet hätte, dass meine Sensoren bereits erkannt wären). In z2m war nichts mehr.
Dass du mich das fragst, habe ich gerade eine Überprüfung gemacht. Mir wird klar, dass ich einen Fehler gemacht haben muss , denn ich dachte, ich hätte zwei z2m-Datenbanken, aber nein, wenn ich zu meinem Gladys-Produktionssystem zurückkehre, habe ich dasselbe z2m mit nur den beiden Sensoren, die ich gerade getestet habe, und es gibt ein rotes Kreuz zwischen Gladys und MQTT in der Konfiguration.
Ich werde warten, bis ich die Tests für deine Entwicklung abgeschlossen habe, und dann werde ich es später sauber aufräumen.
Will_71
1. Dezember 2023 um 12:22
22
Wie hast du den Ordner benannt, der die Datenbank und die Konfiguration enthält?
Kannst du zur Seite von zigbee2mqtt gehen und einen Screenshot des Status-Tabs machen, damit ich alle Exposes deines Sensors sehen kann?
Hier ein Beispiel.
bab85
1. Dezember 2023 um 12:43
23
Für den Ordnernamen musste ich /var/lib/gladysassistant_OWON_test: /var/lib/gladysassistant\ angeben.
Und für den Status habe ich dies:
Will_71
1. Dezember 2023 um 13:02
24
Ok, wenn du den Namen richtig geändert hast, dann ist das nicht die gleiche Konfiguration wie in deiner Produktion, also ist es eine neue Instanz.
Für den Tamper-Export ist er tatsächlich vorhanden, ich schaue mir das heute Abend an und halte dich auf dem Laufenden.
Will_71
1. Dezember 2023 um 22:31
25
@bab85 ,
ich habe herausgefunden, warum du die Funktion „tamper“ auf deinem Sensor nicht gesehen hast, mein Fehler
Ich teste es nochmal auf meiner Seite und werde ein neues Image erstellen.
Ich sage dir Bescheid, wenn es verfügbar ist.
Ich gebe dir gleichzeitig die Befehle, um das Image auf deiner Seite zu aktualisieren.
@pierre-gilles , ich habe auf meiner Seite einen Fehler gemacht beim Versuch, meinen Branch zu rebasen, wodurch die PR geschlossen wurde.
Daher die neue PR
master ← William-De71:zigbee2mqtt-add-missing-exposes-OWON-PIR313-E
ouvert 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

Will_71
1. Dezember 2023 um 23:06
26
Das Bild ist bereit
willde71/gladys-test:zigbee2mqtt-owon-pir313-e
Hier sind die Befehle, falls du es nochmal testen möchtest
docker stop gladys
docker rm gladys
docker pull willde71/gladys-test:zigbee2mqtt-owon-pir313-e
docker run .....
Für die Befehle stop und rm verwende den Namen, den du während deines Tests verwendet hast.
Danach führe den Befehl docker run... erneut aus.
bab85
2. Dezember 2023 um 08:11
27
Hallo,
Ich habe das neue Bild getestet. Perfekt, alles funktioniert einwandfrei. Der Sabotage-Erkennungs-Item ist verfügbar. Wenn der Detektor auf seiner Halterung ist, haben wir den Wert « nein » in grün, andernfalls « ja » in rot. Bei der Batterie funktioniert es weiterhin.
Will_71
2. Dezember 2023 um 13:16
28
@bab85 , danke für deinen Test.
bab85
3. Dezember 2023 um 18:24
29
Vielen Dank vor allem für die Entwicklung und die Schnelligkeit der Berücksichtigung.
Danke für das PR @Will_71 !
Ich habe dir Feedback gegeben:
master ← William-De71:zigbee2mqtt-add-missing-exposes-OWON-PIR313-E
ouvert 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

Will_71
7. Dezember 2023 um 13:22
31
Ich habe gesehen, ich mache die Änderung heute Abend.
Will_71
7. Dezember 2023 um 22:20
32
@pierre-gilles , ich habe die Änderung vorgenommen
Danke für die Fixes @Will_71 ! Getestet und für mich okay
Ich habe gemerged und es wird in der nächsten Version von Gladys veröffentlicht!