Service.Lightbulb support about homebridge-domotiga HOT 6 CLOSED

Hey gurtjun (and samfox2),
i implemented that already in my fork of samfox2's repository. I try to keep in parallel with his most recent version, but didn't hat time to implement some changes he made within last weeks. I hope I can do that it in the next weeks, update my some docu, and provide a pull request.

@Samfox2 Seems hard to get in direct contact with you :). I'm not really aware of some dependencies regarding domotica (e.g. your code accept some a "1", sometimes there was "On" or "true" - I unified that in my fork. I implemented some other features as well (like various backend like url, local commands or file interface). I don't use domotica at all, so I can't run a full compatibility test in any way. So... pull request could be hard to accept :)

from homebridge-domotiga.

Sorry, I am quite busy at the moment :-) Would be really cool if you could implement your changes into the plugin.

from homebridge-domotiga.

Hi @nordblick2,

Cool you already implemented the possibility to add a Lightbulb. Not sure if it's going to work with Domotiga though. E.g. you're using the below function call to get a value from Domotiga:

...
self.domotigaGetValue(thisDevice, "brightness", function (error,value) {
...

As far as I'm aware you can only get values by their number, correct me if I'm wrong @Samfox2? In Domotiga they are represented as value1, value2, value3, etc...
So I don't think "brightness" will work as a value. But I'm not that experienced either with Domotiga so I could be wrong...

from homebridge-domotiga.

@gurtjun yes that‘s right, the column of all 3 values have to be defined in config.json

from homebridge-domotiga.

Thanks to the example in the fork of @nordblick2. I've just created pull request #26 that contains the required functionality. Both switching and dimming a light bulb.

from homebridge-domotiga.

Merge it right now, thank you.

from homebridge-domotiga.