-
Notifications
You must be signed in to change notification settings - Fork 82
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 climatization status #527
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #527 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 17 18 +1
Lines 1410 1446 +36
=========================================
+ Hits 1410 1446 +36
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Will this now make the Climate function 1 button (like the lock), instead of having a separate start and stop switch? I assume by reading the status of climatization we can now have a true single climate entity that will stay on after activating and turn off (or manually stopped), like a fan. |
Yes, this is the plan. At least for cars that return the climate state. However the PR for HA is not yet open, so might take a little bit. |
So not sure it's me not doing something right but I don't see any new entities in the HA integration. Is the integration upgrade process automatic when updating HA? I'm on the 2023.5 release but it doesn't seem like the BMW integration is any different |
Thanks for the reminder - open quite a lot of PRs to HA so forgot to actually add one climate status. Maybe I'll find some time next week to implement it. I'll update in the discussion. |
Proposed change
This PR adds (if available on vehicle):
HEATING
,COOLING
,STANDBY
,INACTIVE
,UNKNOWN
)Type of change
Additional information
Checklist