# HomeKit- und Siri-Integration

**URL:** <https://community.gladysassistant.com/t/integration-homekit-et-siri/7493>\
**Category:** Développement\
**Created:** [23. August 2022 um 21:46 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493 "2022-08-23T21:46:48Z")\
**Posts on this page:** 20\
**Page:** 3

<div class="post-metadata">

**Author:** ![GBoulvin](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/gboulvin/32/10006_2.png) [@GBoulvin](https://community.gladysassistant.com/u/GBoulvin)\
**Post date:** [7. November 2022 um 17:19 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/41 "2022-11-07T17:19:35Z")

</div>

Du musst einen Feature-Request stellen, damit das integriert wird. Vorerst nur Schalter, Lampen und Temperaturen (falls ich nichts vergessen habe).

---

<div class="post-metadata">

**Author:** ![bertrandda](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/bertrandda/32/5929_2.png) [@bertrandda](https://community.gladysassistant.com/u/bertrandda)\
**Post date:** [7. November 2022 um 20:24 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/43 "2022-11-07T20:24:40Z")

</div>

Diese Sensoren werden tatsächlich noch nicht von der HomeKit-Integration unterstützt. Ich werde das für die nächste Version notieren, und zögere nicht, einen Feature-Request wie von @GBoulvin empfohlen zu schreiben, damit wir eine Nachverfolgung haben.

---

<div class="post-metadata">

**Author:** ![bertrandda](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/bertrandda/32/5929_2.png) [@bertrandda](https://community.gladysassistant.com/u/bertrandda)\
**Post date:** [13. November 2022 um 22:43 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/44 "2022-11-13T22:43:48Z")

</div>

Nach einigen Tagen der Nutzung habe ich Änderungen vorgenommen, um die Integration zu verbessern. Dieser neue Pull Request

- fügt die Öffnungssensoren (Tür, Fenster…) auf Anfrage von @bouillou7 hinzu
- behebt den Helligkeitsbereich der Lampen. Heute zum Beispiel bei Philips Hue, wenn man die Helligkeit auf 100% über die Maison-App einstellt, liegt man tatsächlich nur bei ~40% bei Gladys
- behebt den Empfang neuer Werte bei HomeKit. Derzeit werden die neuen Werte der Gladys-Geräte nicht direkt vom HomeKit-Netzwerk empfangen, sondern die App muss diese neuen Werte anfordern (beim Starten, beim Wechseln der Ansicht in der App…). Mit dieser Entwicklung sendet Gladys eine „Benachrichtigung“ an das HomeKit-Netzwerk bei jedem neuen Wert eines kompatiblen Geräts, die Anzeige in der Maison-App wird tatsächlich in Echtzeit und dies wird die Automatisierungen der Maison-App verbessern, die von den Sensoren abhängig sind

Für diejenigen, die einen Blick darauf werfen oder testen möchten, der Pull Request:

> <https://github.com/GladysAssistant/Gladys/pull/1650>
>
> \### 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:
> 
> \- \[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\] 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 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
> 
> Add contact sensors (door, window...)
> Fix bulb brightness range
> Notify HomeKit of new value

Ein Docker-Image ist verfügbar `bertrandda/homekit`

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [14. November 2022 um 03:24 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/45 "2022-11-14T03:24:50Z")

</div>

Genial, das! 🙂

Ich habe dir einen Kommentar zur Häufigkeit des Sendens der neuen Zustände hinterlassen. Alle Zustände zu senden, ist wirklich zu aggressiv. Ich habe es bei der Alexa/Google Home-Integration etwas anders gemacht 🙂

---

<div class="post-metadata">

**Author:** ![bertrandda](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/bertrandda/32/5929_2.png) [@bertrandda](https://community.gladysassistant.com/u/bertrandda)\
**Post date:** [15. November 2022 um 20:17 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/46 "2022-11-15T20:17:53Z")

</div>

Bei Google/Alexa ist das der Fall, weil wir API-Aufrufe und Serveranfragen tätigen. Wenn alles lokal bleibt, ist das dann nicht unnötig?

Ich habe mir den Code ein bisschen angesehen. Kannst du mir bitte bestätigen, ob ich die Funktionsweise richtig verstanden habe: Sobald wir ein Ereignis über den neuen Zustand einer Funktion erhalten, starten wir einen Timeout von 5 Sekunden. Wenn innerhalb dieser 5 Sekunden kein neues Ereignis für dieselbe Funktion eintrifft, senden wir die Information. Andernfalls starten wir einen neuen Timeout mit dem neuen Wert.

Wenn das stimmt, habe ich zwei Fragen:

- Was passiert, wenn ein Gerät alle 2 Sekunden automatisch einen neuen Wert sendet (ich habe kein Beispiel, aber ich versuche, extreme Fälle zu berücksichtigen, z. B. mit dem Zigbee-Modul des Linky)? Wird man dann nicht in einer Endlosschleife stecken, ohne den neuen Wert zu senden?
- Können wir den Timeout nicht etwas verkürzen? Denn wenn der Benutzer z. B. einen Alarm/eine Warnung in den HomeKit-Automatisierungen programmiert hat, der/die bei einem Öffnungssensor von Gladys ausgelöst wird, könnte es sein, dass ein Fenster/Tür geöffnet und dann innerhalb von 5 Sekunden wieder geschlossen wird und der Alarm nicht ausgelöst wird. Ist das nicht etwas, das wir aus Sicherheitsgründen vermeiden wollen?

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [18. November 2022 um 06:47 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/47 "2022-11-18T06:47:05Z")

</div>

> [@bertrandda](#):
>
> Was passiert, wenn ein Gerät alle 2 Sekunden automatisch einen neuen Wert sendet (ich habe kein Beispiel, aber ich versuche, extreme Fälle zu imaginieren, z. B. mit dem Zigbee-Modul des Linky)? Wird man dann nicht in einer Endlosschleife landen, ohne den neuen Wert jemals zu senden?

Genau das habe ich mir auch gedacht, als ich dir den Code-Schnipsel geschickt habe 😛 Das ist ein Bug, es sollte eine Art «&nbsp;Maximalverzögerung&nbsp;» geben.

> [@bertrandda](#):
>
> Im Fall von Google/Alexa ist das so, weil wir API-Aufrufe an ihre Server machen. Wenn es lokal bleibt, ist das dann nicht unnötig?

Das erzeugt wirklich enormen Traffic, und wir haben das bei der Entwicklung der InfluxDB-Integration gesehen: Es verbraucht wirklich viel CPU auf Gladys-Seite, jeden einzelnen Event ohne irgendwelche Grenzen zu verwalten.

Selbst wenn die Grenze bei 1 Event pro Sekunde liegt, ist das schon super. Manche Geräte sind sehr ausführlich.

> [@bertrandda](#):
>
> Kann man den Timeout nicht etwas reduzieren? Denn wenn der Benutzer z. B. einen Alarm/eine Warnung in den HomeKit-Automatisierungen programmiert hat, der bei einem Gladys-Öffnungssensor ausgelöst wird, könnte es sein, dass ein Fenster/Tür geöffnet und dann innerhalb von 5 Sekunden wieder geschlossen wird und der Alarm nicht ausgelöst wird. Ist das nicht etwas, das wir aus Sicherheitsgründen vermeiden wollen?

In diesem Fall können wir die Verzögerung tatsächlich reduzieren. Danach kann die Verzögerung für bestimmte Arten von «&nbsp;kritischen&nbsp;» und wenig ausführlichen Geräten (z. B. Türöffnung) unterschiedlich sein.

Aber ich versichere dir, dass bei den Installationen einiger Benutzer dieses Forums das Senden von allem ihr Netzwerk und ihre Gladys-Instanz töten wird 😅

Eine andere Frage: Du sendest nur die Zustände der Geräte, die «&nbsp;Gladys-Homekit-kompatibel&nbsp;» sind, oder?

---

<div class="post-metadata">

**Author:** ![bertrandda](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/bertrandda/32/5929_2.png) [@bertrandda](https://community.gladysassistant.com/u/bertrandda)\
**Post date:** [19. November 2022 um 11:00 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/48 "2022-11-19T11:00:51Z")

</div>

> [@pierre-gilles](#):
>
> Tatsächlich habe ich mir beim Senden des Code-Schnipsels dasselbe gedacht 😛 Es ist ein Bug, es sollte eine Art „Maximal-Tempo“ geben.

Okay, ich werde es versuchen, indem ich das Doppelte als Maximum einstelle (wenn die Tempo 5 Sekunden beträgt und wir in einem Fall sind, in dem ein Ereignis alle 2 Sekunden auftritt, wird der neue Wert mit dem Ereignis gesendet, das 10 Sekunden nach dem ersten auftritt)

> [@pierre-gilles](#):
>
> In diesem Fall können wir die Tempo tatsächlich reduzieren. Allerdings kann die Tempo für bestimmte Arten von „kritischen“ und wenig ausführlichen Geräten unterschiedlich sein (z. B. Türöffnung)

Standardmäßig werde ich also 5 Sekunden einstellen und im Service-Mapping ein Feld hinzufügen, um diese Verzögerung bei Bedarf anzupassen

> [@pierre-gilles](#):
>
> Eine andere Frage: Du sendest nur die Zustände der Geräte, die „Gladys-Homekit-kompatibel“ sind, oder?

Ja, nur die neuen Werte der Geräte, die an HomeKit angeschlossen sind, werden gesendet

---

<div class="post-metadata">

**Author:** ![bertrandda](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/bertrandda/32/5929_2.png) [@bertrandda](https://community.gladysassistant.com/u/bertrandda)\
**Post date:** [24. November 2022 um 19:23 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/49 "2022-11-24T19:23:13Z")

</div>

Ich habe das Bild `bertrandda/gladys:homekit` mit den neuesten Änderungen aktualisiert, um die Anzahl der gesendeten Ereignisse zu reduzieren.

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [25. November 2022 um 06:46 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/50 "2022-11-25T06:46:25Z")

</div>

Vielen Dank @bertrandda für das Fix 🙂

Jetzt Feedback einholen, um zu prüfen, ob es funktioniert und nicht zu aggressiv ist. Wenn ihr testet, wäre ein Vergleich „CPU-Nutzung vor/nach“ super.

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [20. Dezember 2022 um 11:41 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/51 "2022-12-20T11:41:13Z")

</div>

Ein kleines Up für alle, die dieses Thema während der Ferien testen können 🙂

Ansonsten @bertrandda, nichts damit zu tun, aber wir haben einen Typen im englischen Forum, der Fragen zu Node-HAP stellt (kein Bezug zu Gladys), falls du ihm mal helfen möchtest:

[https://en-community.gladysassistant.com/t/we-are-trying-to-use-hap-nodejs-we-are-able-to-control-devices-from-home-app-but-when-we-control-the-devices-manually-the-status-is-not-reflecting-in-home-app-can-you-please-help-us-on-this/65](https://en-community.gladysassistant.com/t/we-are-trying-to-use-hap-nodejs-we-are-able-to-control-devices-from-home-app-but-when-we-control-the-devices-manually-the-status-is-not-reflecting-in-home-app-can-you-please-help-us-on-this/65)

---

<div class="post-metadata">

**Author:** ![bertrandda](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/bertrandda/32/5929_2.png) [@bertrandda](https://community.gladysassistant.com/u/bertrandda)\
**Post date:** [20. Dezember 2022 um 18:59 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/52 "2022-12-20T18:59:53Z")

</div>

> [@pierre-gilles](#):
>
> Übrigens @bertrandda, das hat nichts damit zu tun, aber im englischen Forum gibt es einen Typen, der Fragen zu Node-HAP stellt (keine Verbindung zu Gladys), falls du ihm helfen möchtest

Ich schaue mal, was ich tun kann

---

<div class="post-metadata">

**Author:** ![bertrandda](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/bertrandda/32/5929_2.png) [@bertrandda](https://community.gladysassistant.com/u/bertrandda)\
**Post date:** [4. Januar 2023 um 19:14 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/53 "2023-01-04T19:14:34Z")

</div>

Ich habe das Bild aktualisiert, um jetzt Node 18 zu verwenden. Zur Erinnerung ermöglicht das Bild `bertrandda/gladys:homekit` das Testen der folgenden Änderungen:

- Echtzeitdaten in HomeKit empfangen
- Helligkeit der Lampen korrigiert
- Verwaltung von Öffnungssensoren hinzugefügt.

---

<div class="post-metadata">

**Author:** ![cicoub13](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/cicoub13/32/4482_2.png) [@cicoub13](https://community.gladysassistant.com/u/cicoub13)\
**Post date:** [10. Januar 2023 um 16:48 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/54 "2023-01-10T16:48:23Z")

</div>

Hallo,

Ich habe diesen Fehler in den Logs

```auto
2023-01-10T17:45:47+0100 <warn> functionsWrapper.js:15 (EventEmitter.<anonymous>) Fehler beim Ausführen der Funktion () { [native code] }
2023-01-10T17:45:47+0100 <warn> functionsWrapper.js:16 (EventEmitter.<anonymous>) TypeError: Cannot read properties of undefined (reading 'notifDelay')
    at HomeKitHandler.notifyChange (/src/server/services/homekit/lib/notifyChange.js:23:71)
    at EventEmitter.<anonymous> (/src/server/utils/functionsWrapper.js:13:13)
    at EventEmitter.emit (node:events:525:35)
    at Event.emit (/src/server/lib/event/index.js:18:16)

```

et diesen Fehler auf dem iPad, wenn ich versuche, Gladys hinzuzufügen (obwohl ich sie bereits gelöscht habe)

 ![IMG_E365FC6CC513-1](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/9/90802448f0e8fa74c904831c864a63fe718ae760.jpeg)

Nach dem Löschen der Variablen in t\_variables in der Datenbank konnte ich Gladys neu verbinden.  
Die Öffnungssensoren funktionieren (und können als Türen angezeigt werden).

Aber immer noch die gleichen Fehler in Gladys und der Zustand der Öffnungssensoren ändert sich nicht in HomeKit (er scheint übrigens invertiert zu sein)

 ![IMG_361C8C48D82D-1](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/6/636ddd745014337c3cea7fb68ac239435e0a7ced.jpeg)

---

<div class="post-metadata">

**Author:** ![bertrandda](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/bertrandda/32/5929_2.png) [@bertrandda](https://community.gladysassistant.com/u/bertrandda)\
**Post date:** [12. Januar 2023 um 21:47 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/55 "2023-01-12T21:47:00Z")

</div>

Vielen Dank @cicoub13 für deine Tests. Das zeigt die Grenzen der Entwicklungsumgebung mit falschen Geräten.

> [@cicoub13](#):
>
> Ich habe diesen Fehler in den Logs

Ich glaube, ich weiß, was passiert ist. Kannst du mir bitte die Referenz deines Türsensors geben?

> [@cicoub13](#):
>
> Er scheint übrigens invertiert zu sein

Ich werde diesen Punkt überprüfen

> [@cicoub13](#):
>
> Nach dem Löschen der Variablen in t\_variables in der Datenbank konnte ich Gladys neu zuordnen.  
> Die Öffnungssensoren funktionieren gut (und man kann sie als Türen anzeigen).

In den HomeKit-Foren habe ich viele ähnliche Fälle gesehen. Es scheint, dass der Löschbefehl nicht immer bis zum Gerät durchdringt, sodass das Gerät aus deiner Home-App verschwindet, aber nicht aus der internen Konfiguration des Geräts. Ich werde einen `zurücksetzen`-Button auf der Integrationsseite hinzufügen, um den Konfigurationsordner zu löschen. Das sollte es ermöglichen, nicht in die Tabelle eingreifen zu müssen, wenn man diese Meldung erhält.

---

<div class="post-metadata">

**Author:** ![cicoub13](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/cicoub13/32/4482_2.png) [@cicoub13](https://community.gladysassistant.com/u/cicoub13)\
**Post date:** [13. Januar 2023 um 12:03 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/56 "2023-01-13T12:03:54Z")

</div>

> [@bertrandda](#):
>
> Ich glaube, ich weiß, was passiert ist. Kannst du mir bitte die Referenz deines Türsensors geben?

Dieser hier: [Aqara MCCGQ11LM control via MQTT | Zigbee2MQTT](https://www.zigbee2mqtt.io/devices/MCCGQ11LM.html#xiaomi-mccgq11lm)

> [@bertrandda](#):
>
> Ich werde einen `Zurücksetzen`-Button auf der Integrationsseite hinzufügen, um den Konfigurationsordner zu löschen. Das sollte verhindern, dass man in die Tabelle gehen muss, wenn man diese Meldung erhält.

Das ist eine gute Idee, aber man muss gut erklären, was der Button macht (und ihn vielleicht rot einfärben).  
Denn es gibt bereits einen Button, wenn die Hinzufügung eines Geräts in HomeKit nicht funktioniert, um den «&nbsp;Dienst neu zu laden&nbsp;».

---

<div class="post-metadata">

**Author:** ![bertrandda](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/bertrandda/32/5929_2.png) [@bertrandda](https://community.gladysassistant.com/u/bertrandda)\
**Post date:** [15. Januar 2023 um 19:54 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/57 "2023-01-15T19:54:36Z")

</div>

Ich habe dir ein Bild mit dem Fix für den invertierten Wert und das Senden des Live-Werts erstellt. Kannst du mir sagen, ob es okay ist?

---

<div class="post-metadata">

**Author:** ![cicoub13](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/cicoub13/32/4482_2.png) [@cicoub13](https://community.gladysassistant.com/u/cicoub13)\
**Post date:** [20. Januar 2023 um 08:49 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/58 "2023-01-20T08:49:48Z")

</div>

Alles klar für mich, danke für die Korrektur

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [23. Januar 2023 um 04:28 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/59 "2023-01-23T04:28:59Z")

</div>

Alles klar bei mir, nur eine Frage vor dem Merge: Kann ich das ohne Bedenken in die Produktion deployen? Der Nutzer muss nichts neu verknüpfen, oder? 🙂

---

<div class="post-metadata">

**Author:** ![bertrandda](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/bertrandda/32/5929_2.png) [@bertrandda](https://community.gladysassistant.com/u/bertrandda)\
**Post date:** [23. Januar 2023 um 17:19 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/60 "2023-01-23T17:19:02Z")

</div>

Normalerweise gilt Gladys als Brücke, neue Geräte werden automatisch zu HomeKit hinzugefügt. Man muss nur die Namen/Räume der neuen Geräte konfigurieren (sie haben standardmäßig einen Raum, bis dies erledigt ist).

Das Problem, über das wir mit Cyril gesprochen haben, ist ein Sonderfall, den ich in einem anderen PR behandeln werde. Es muss der Fall verwaltet werden, in dem der Benutzer versucht, Gladys von HomeKit zu trennen, aber er zu diesem Zeitpunkt nicht im Netzwerk erreichbar ist. Auf der Seite der Maison Gladys-App wird Gladys vergessen, aber auf der Seite von Gladys weiß es nicht, dass die Verbindung gelöst werden muss, daher ist ein Reset erforderlich (siehe [FAQ · homebridge/HAP-NodeJS Wiki · GitHub](https://github.com/homebridge/HAP-NodeJS/wiki/FAQ#how-to-perform-a-factory-reset-of-the-accessory)).

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [25. Januar 2023 um 13:33 UTC](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493/61 "2023-01-25T13:33:42Z")

</div>

Alles klar, Nickel!

Ich habe gerade etwas im Code gesehen, ich habe dir hier einen Kommentar hinterlassen:

> <https://github.com/GladysAssistant/Gladys/pull/1650/files#r1086645875>
>
> \### 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:
> 
> \- \[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\] 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 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
> 
> Add contact sensors (door, window...)
> Fix bulb brightness range
> Notify HomeKit of new value

[Vorherige Seite](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493.md?page=2)

[Nächste Seite](https://community.gladysassistant.com/t/integration-homekit-et-siri/7493.md?page=4)
