hass-template-climate icon indicating copy to clipboard operation
hass-template-climate copied to clipboard

turn_on/turn_off methods without setting the proper ClimateEntityFeature

Open purcell-lab opened this issue 1 year ago • 4 comments

The problem

Home Assistant 2024.3 is currently in beta and is doing some more strict checking of climate devices and has detected an issue with this integration.

What version of Template Climate has the issue?

0.6.1

What version of Home Assistant are you running?

2024.3.b1

What type of installation are you running?

Home Assistant Supervised

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Logger: homeassistant.components.climate
Source: components/climate/__init__.py:361
integration: Climate (documentation, issues)
First occurred: 5:30:03 AM (3 occurrences)
Last logged: 5:32:51 AM

Entity climate.climate_master (<class 'custom_components.climate_template.climate.TemplateClimate'>) implements HVACMode(s): auto, off, cool, heat, dry, fan_only and therefore implicitly supports the turn_on/turn_off methods without setting the proper ClimateEntityFeature. Please create a bug report at https://github.com/jcwillox/hass-template-climate/issues

Additional information

No response

purcell-lab avatar Feb 29 '24 20:02 purcell-lab

Hello,

if you like, you can use my forked repo, where I am merging multiple feature request with some of my own patches. ;-) The above error is patched there.

litinoveweedle avatar Apr 06 '24 19:04 litinoveweedle

just create a pr to this repo

SiriosDev avatar Aug 23 '24 22:08 SiriosDev

Why should I? I am happy to maintain my fork, which has now more features, some of those were long PR in this repo but were never merged. There has to be two to dance. So thank you, but no thank you.

litinoveweedle avatar Aug 23 '24 22:08 litinoveweedle

Dunno, (for me) it doesn't seem the most elegant thing in the world to me to fork without even trying a merge, when the original repo is still maintained.

That doesn't mean I won't switch to your version which seems better. You seem sure you don't want to merge, which is the second most important thing after functions (not having to switch back, in a word stability)

SiriosDev avatar Aug 23 '24 23:08 SiriosDev