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

Merge dev changes into main ready for 1.2.0 release #33

Merged
merged 8 commits into from
Apr 15, 2024
Merged

Conversation

tonyroberts
Copy link
Owner

No description provided.

tonyroberts and others added 8 commits March 2, 2024 18:07
Only have samples from HW 2 and 4 for now. All other versions will
log a warning and use the same ranges as for hw 4 (since that doesn't
change the current behaviour for other versions).

Fixes #21
Add dev branch to tests github workflow
* Fix deprecates uses of TEMP_CELSIUS

* Fix warning about ClimateEntityFeature usage

This adds ClimateEntityFeature.TURN_ON and ClimateEntityFeature.TURN_OFF
to supported features and disables the backwards compatibility flag
related to those. This prevents a warning at initialization.

Specific warning is:
Entity None (<class 'custom_components.wundasmart.climate.Device'>)
implements HVACMode(s): off, auto, heat and therefore implicitly
supports the turn_on/turn_off methods without setting the proper
ClimateEntityFeature. Please create a bug report at
https://github.com/tonyroberts/hawundasmart/issues

* Remove TURN_OFF and TURN_ON flags
When turning on, set temp to the comfort preset temp instead of the
current temp + 1.
State and operation_mode are the same thing for the WaterHeaterEntity,
and the current operation mode (which self.state returns) has to be an
operation mode that can also be selected.

When a manual override has been set try to determine the most sensible
operation mode to display. Otherwise, use on/off and add those to the
list of operation modes allowed. Since we can't turn the water heater on
or off without a duration, alias on/off to the longest preconfigured
boost/off modes.

Fixes #27
Instead of just preventing multiple commands from being sent at once,
prevent multiple sessions from being used at the same time.

This should prevent the coordinator from polling new values while a
command is being sent.

Fixes #28
@tonyroberts tonyroberts merged commit 9e69366 into main Apr 15, 2024
6 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