-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[daikin] Add demand control support for ac_unit #17087
Conversation
5a95d70
to
556c994
Compare
ac_unit
Thing.Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
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.
Thanks, LGTM. Would be nice if @caffineehacker can comment.
AFAIK, we haven't heard from @caffineehacker about this binding in years (at least 4 years that I know of) |
He seems active on github, maybe not getting these notification? Or if he abondoned this binding, it would be nice to get a comment from him and we can remove him as codewoner. Ping @caffineehacker |
Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au> Signed-off-by: Patrik Gfeller <patrik.gfeller@proton.me>
Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
Resolve #16479 and #17036