ilcato / homebridge-fibaro-home-center

Homebridge plugin for Fibaro Home Center
Apache License 2.0
28 stars 24 forks source link

new heating zones missing #309

Closed tzalexander closed 8 months ago

tzalexander commented 8 months ago

I have created a new heating section after the homebridge fibaro plugin setup. I don't get it into the accessories list. How do I get a new heating control section into that list? Thank you

ilcato commented 8 months ago

@tzalexander, which gateway are you using (HC3, HC2, Yubii, ...)?

tzalexander commented 8 months ago

@ilcato I am using HC2

ilcato commented 8 months ago

@tzalexander can you try to call the '/api/panels/heating' API and see if all the heating zones are returned?

tzalexander commented 8 months ago

@ilcato yes, all heating zones are returned correctly

tzalexander commented 8 months ago

@ilcato is it possible that the heating ids somehow collides with device ids?

ilcato commented 8 months ago

@tzalexander interesting

ilcato commented 8 months ago

@tzalexander do you have such a collision in your setup?

ilcato commented 8 months ago

@tzalexander can you test 1.7.4-beta.2 ?

tzalexander commented 8 months ago

@ilcato collision? I checked all heating IDs in api/panels/heating (6, 511, 512, 513, 514). Then i used these IDs at api/devices/ID.

@ilcato test 1.7.4-beta.2? That was a BIG mistake :( . The cli ran full of the following error and all devices in homekit are gone (cry):

Error: This callback function has already been called by someone else; it can only be called one time.
    at /var/lib/homebridge/node_modules/homebridge/node_modules/hap-nodejs/src/lib/util/once.ts:12:13
    at FibaroAccessory.getCharacteristicValue (/var/lib/homebridge/node_modules/homebridge-fibaro-home-center/src/fibaroAccessory.ts:527:5)
    at BatteryLevel.<anonymous> (/var/lib/homebridge/node_modules/homebridge-fibaro-home-center/src/fibaroAccessory.ts:498:14)
    at BatteryLevel.emit (node:events:530:35)
    at /var/lib/homebridge/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:2463:14
    at new Promise (<anonymous>)
    at BatteryLevel.<anonymous> (/var/lib/homebridge/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:2461:12)
    at step (/var/lib/homebridge/node_modules/homebridge/node_modules/tslib/tslib.js:195:27)
    at Object.next (/var/lib/homebridge/node_modules/homebridge/node_modules/tslib/tslib.js:176:57)
    at /var/lib/homebridge/node_modules/homebridge/node_modules/tslib/tslib.js:169:75
ilcato commented 8 months ago

@tzalexander remove from the cache all heating zone related accessories.

ilcato commented 8 months ago

@tzalexander try beta 3

tzalexander commented 8 months ago

@ilcato the update to beta 3 worked fine. all heating zones are listed in the accessories list now. thank you