NorthernMan54 / homebridge-dht

Homebridge plugin for DHT22 Temperature / Humidity Sensors
72 stars 14 forks source link

Recent homebridge update broken things? #10

Closed SkyJohn closed 6 years ago

SkyJohn commented 6 years ago

The recent update to home bridge (0.4.25) seems to have broken this, homebridge wont start with it installed.

Similar issue to this I think: https://github.com/nfarina/homebridge/issues/1433#issuecomment-322804542

pdlove commented 6 years ago

As a note over here... Some plugins had issue because Apple changed some characteristics on services with the IOS 11 releases. I've fixed HAP-NodeJS to simply give warnings when these are encountered. For the time being Apple isn't rejecting them, but you never know when they might start. Let me know if you need more information on how to resolve this.

NorthernMan54 commented 6 years ago

@pdlove

Tks for the warning on the changes, I should be okay with making the fixes for this, but I will let you know if I need an assist. I have been watching these changes for IOS 11 closely, as I maintain a forked version of hap-nodesjs that supports Alexa. And I expect some challenges with accessory numbering as a result of your

NorthernMan54 commented 6 years ago

@pdlove

Updating via mobile, and it went and closed it

Changes to aid and iid.

pdlove commented 6 years ago

The changes I made are only to the iids on the bridge and only when there is more than one service. It shouldn’t cause you any issues.

NorthernMan54 commented 6 years ago

I found the issue, I was setting the serial number to "undefined". It has been fixed and promoted. Please update and try again

SkyJohn commented 6 years ago

Works now :)

Thanks for the quick update.