Add support for Velux light devices#49338
Conversation
|
Hi @bramstroker, It seems you haven't yet signed a CLA. Please do so here. Once you do that we will be able to review and accept this pull request. Thanks! |
|
Hey there @Julius2342, mind taking a look at this pull request as its been labeled with an integration ( |
|
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. |
|
Not stale |
|
How long does it usually take for a pull request like this to be merged into home assistant? |
e0acb40 to
44d5bb2
Compare
44d5bb2 to
787898b
Compare
Co-authored-by: Franck Nijhof <frenck@frenck.nl>
Co-authored-by: Franck Nijhof <frenck@frenck.nl>
|
Rebased the PR and resolved latest comments. And did some final testing. Works on my machine ;-) |
|
pipeline failed, should be fixed now hopefully. |
Proposed change
I have a Velux KLF 200 with 4 windows, 4 blinds and 4 led spots connected. The integration was missing support to control my lights. This PR adds support
Type of change
Checklist
black --fast homeassistant tests)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest.requirements_all.txt.Updated by running
python3 -m script.gen_requirements_all..coveragerc.The integration reached or maintains the following Integration Quality Scale:
To help with the load of incoming pull requests: