Skip to content
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 ClimateEntityFeature.TURN_OFF feature & implementation #176

Merged
merged 6 commits into from
Dec 15, 2024

Conversation

mxr
Copy link
Contributor

@mxr mxr commented Dec 13, 2024

This is a followup of #163 which implements the turn_off action and updates documentation.

@mxr mxr changed the title Add ClimateEntityFeature.TURN_OFF and ClimateEntityFeature.TURN_OFF feature flags Add ClimateEntityFeature.TURN_OFF and ClimateEntityFeature.TURN_ON feature flags Dec 13, 2024
@mxr mxr changed the title Add ClimateEntityFeature.TURN_OFF and ClimateEntityFeature.TURN_ON feature flags Add ClimateEntityFeature.TURN_OFF and ClimateEntityFeature.TURN_ON features Dec 13, 2024
@mxr mxr changed the title Add ClimateEntityFeature.TURN_OFF and ClimateEntityFeature.TURN_ON features Add ClimateEntityFeature.TURN_OFF and ClimateEntityFeature.TURN_ON features & implementation Dec 13, 2024
custom_components/kumo/climate.py Outdated Show resolved Hide resolved
custom_components/kumo/climate.py Outdated Show resolved Hide resolved
@mxr mxr changed the title Add ClimateEntityFeature.TURN_OFF and ClimateEntityFeature.TURN_ON features & implementation Add ClimateEntityFeature.TURN_OFF feature & implementation Dec 14, 2024
@mxr mxr requested a review from dlarrick December 14, 2024 14:43
@dlarrick
Copy link
Owner

Limiting the scope here for compatibility with 2025.1 seems like a good idea. Thanks.

Given how much back & forth there's been on this one I want to try it first in my own system before merging, but right now this looks good. I hope to find time this weekend to catch up on a few things with the integration.

I'd also accept a separate PR that adds turn_on capability by storing the last-used climate mode in a small JSON file.

@mxr
Copy link
Contributor Author

mxr commented Dec 14, 2024

Sounds good. It worked on my system so hopefully it works in yours as well.

@dlarrick
Copy link
Owner

Addresses part of #164

@dlarrick dlarrick merged commit 7d559b0 into dlarrick:master Dec 15, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants