snowdd1 / homebridge-knx

KNX platform shim for homebridge
https://github.com/nfarina/homebridge
GNU General Public License v2.0
97 stars 55 forks source link

Issues with npm-knx #162

Closed Luehrsen closed 4 years ago

Luehrsen commented 4 years ago

Now this one was hard to figure out. 😄

On the current master branch and due to the excellent work by Paul, the existing implementations based on knxd WILL break. Existing users will have to update the knx_config.json and add "knxd: true" to the root of the json.

This might lead to increased support issues. You might want to leave the default case as is and make the new way the optin way.

I noticed that because the communication from knx to hk broke. No messages from the bus arrived in homebridge-knx. The other way around was fine. Means: I was able to switch lights from HK, but when I switched lights in KNX, the updated state was not reflected in HK.

snowdd1 commented 4 years ago

You are right. Will change that to force "knxconnection": "knxjs" for pure knx js to use.

snowdd1 commented 4 years ago

fixed with https://github.com/snowdd1/homebridge-knx/commit/daec87f9d60a72f4aef414afd771011850e613f3

snowdd1 commented 4 years ago

... in version 0.3.24 in npm.