search

benzman81 / homebridge-nukiio

Nuki.io support for Homebridge: https://github.com/nfarina/homebridge
GNU General Public License v3.0
42 stars 6 forks source link

Add polling feature and fix for homekit notifications #45

Closed nicoh88 closed 5 years ago

nicoh88 commented 5 years ago

Issue #39 Issue #43 (only one point, not "different rooms")

nicoh88 commented 5 years ago

@benzman81 Tested! Works great! Please merge. 👍

benzman81 commented 5 years ago

Currently on vacation, but polling the lock state is a very bad idea as this consumes battery everytime it polls. This is why nuki implemented callbacks which work great!

nicoh88 commented 5 years ago

Hi Markus,

ah okay. 👎

~~Bei mir, mit Firmware 2.5.2, funktionieren die "Callbacks" nicht so gut - daher habe ich das Polling eingebaut. Oftmals wird der Status, wenn dieser über die Nuki-App oder händisch geändert wird, in HomeKit nicht geändert. Dies betrifft nicht nur dein Plugin, sondern auch den nativen HomeKit-Support. Hoffentlich wird es dann mit dem nächsten Firmware-Update stabiler.~~

webhook_server_ip_or_name und webhook_port hatte ich bisher nicht konfiguriert. MEIN FEHLER!

Ich erstelle bzgl. der HomeKit-Mitteilungen einen neuen PR.

Gruß Nico