Sese-Schneider / ha-cover-time-based

Integration which allows cover control based on time.
MIT License
6 stars 5 forks source link

Add support for existing cover entities #15

Open TheYves opened 3 weeks ago

TheYves commented 3 weeks ago

Hi there

I've been using cover-rf-time-based because it allowed me to simply use an existing cover entity instead of using switches for the open/close/stop actions. Since this repository has been archived, I'm curious if it would make sense to add a new configuration option "cover_entity_id" to this project and then directly use the corresponding services/actions from the cover entity. As far as I can tell, besides the documentation and configuration, it's just this part that is currently missing.

Use case: I have a third party integration for all my covers in my house, which doesn't provide any state for the cover. So I already have 10 cover entities and all I had to do, is wrapping them with "cover-rf-time-based" and configure the corresponding travel times. Using this integration, I first need to create 30 switches (open/close/stop for 10 entities) in order to get it to work. Or am I missing something?

So if this would be a useful enhancement for this integration and someone would like to implement this, I would love to sponsor a beer/coffee or two. 😊 Otherwise, I have to do it myself, but I'm not very familiar with python, so I would prefer the first option. 😊

Sese-Schneider commented 2 weeks ago

Hey @TheYves This sounds like a usefull addition, I try to come up with a pre-release which implements this for you to try out in the next few days.