-
Notifications
You must be signed in to change notification settings - Fork 21
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
add wakeup for the Linak DPG1C controller #370
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I also propose to add self.wakeup() inside async def connect(self):
26bec6d
to
4b5e457
Compare
@abmantis if you have time your input here would be great since you maintain the HASS component. Do you know of any problems with this approach? I tried this with my unmodified desk/controller and found no issues sending the wakeup command, but it's always good to double check :) |
Sorry for the delay in replying @newAM ! I will try to test it over the weekend. If you are fine with bumping the major version of the lib, than feel free to merge this as it will not affect HA for now! Later, if I find any issues during tests we can work on them :) |
Hi @abmantis, I was just wondering if you had time yet to test this to see if it can be added into HA? |
Yes, it worked fine on my desk (not DPG1C) and should be released with HA 2024.2! |
Ok perfect, thank you! |
CC #369