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

Remove auxiliary heating #81

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

robigan
Copy link
Contributor

@robigan robigan commented Sep 28, 2024

Resolves the error
vimar::VimarClimate implements the is_aux_heat property or uses the auxiliary heater methods in a subclass of ClimateEntity which is deprecated and will be unsupported from Home Assistant 2024.10. Please create a bug report at https://github.com/h4de5/home-assistant-vimar/issues

Opening PR to discuss how to implement boost heating. Home assistant docs say that we should implement the Aux heat functionality as a switch part of the device

@robigan
Copy link
Contributor Author

robigan commented Sep 28, 2024

Resolves the error vimar::VimarClimate implements the is_aux_heat property or uses the auxiliary heater methods in a subclass of ClimateEntity which is deprecated and will be unsupported from Home Assistant 2024.10. Please create a bug report at https://github.com/h4de5/home-assistant-vimar/issues

Opening PR to discuss how to implement boost heating. Home assistant docs say that we should implement the Aux heat functionality as a switch part of the device

This would most likely require to create a new entity class that subclasses ToggleEntity (and of course VimarEntity), although as for how to register that toggle, I'm not sure and your input @h4de5 would be greatly appreciated. I reckon that the ClimateEntity should conditionally register this new entity, but I'm not sure about how to do that or how to handle cases in the future where the entity might not be available, etc.

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.

1 participant