-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for switches as lights #119
Comments
This is a feature that I also wanted to add. I just haven't needed it yet ;-) I'm still not quite sure what the big difference is between a This feature should not be too difficult to implement. The name First of all, a config option is required: Then a property method: In the publish method of After that a |
I'm still trying to remember and I think that main problem is that I'm not able to do turn on all lights without this. No promises, but I'll try to take a look into this. |
I'm busy with another project at the moment but I can look at this issue as soon as I have time again. If you have time, the issue #122 would be better for you, I can't do it alone because I don't have the right hardware. |
Hi,
I discovered your project yesterday when I tried to replace some of custom scripts I'm using on my Unipi. After some digging, it seems it is much much better then I hoped for, thanks for it!
I'm currently using modbus integration in HA, but unipi-control seems much faster (more responsive), so I would like to ditch all my modbus configs and switch over to it. But I realized that all RO's are discovered as switches and there is no option in
Features
to handle RO aslight
instead ofswitch
.TBH, I'm not 100 % sure at this moment why I need light switches as lights :) but I remember I had troubles with this in HA before and it had been worth it to change modbus switches to lights.
I'm opening this issue more as a discussion - are you open to have this feature in unipi-control? If yes, I can try to prepare PR, but I would appreciate any tips for where to begin and how you would like to have this configured. My thought was add something like
device_integration
next todevice_class
.The text was updated successfully, but these errors were encountered: