sfalkman / ngenic-hass-platform

Ngenic platform for Home Assistant
MIT License
33 stars 18 forks source link

Support rooms with activeControl property #31

Closed parherman closed 3 years ago

sfalkman commented 3 years ago

Looks good, but I don't understand the usecase for this.
Can you have multiple sensors that can control the tune? I can't find any documentation for the activeControl attribute.

parherman commented 3 years ago

No the docs are update with information about activeControl: https://developer.ngenic.se/

Certain setups allow multiple sensors to control the tune.

Other setups have no temp sensors at all, and neither roomToControlUuid or activeControl will be set.

sfalkman commented 3 years ago

Hey, There's really no way for me to test this. Have you tested with and without activeControl set? If you have, I will merge the request.

EDIT: Tested with my own setup and it works, merging this.

parherman commented 3 years ago

Sorry for the late response!

Yes I have tested it with a roomToControl and a activeControl system

Thanks for merging this

kungktd commented 2 years ago

@parherman How do you choose which room that should control the temperature? Cant find it anywhere in HA. But my knowledge with this is basic, so I’m not surprised. :)