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
Allow setting hvac mode through set_temperature climate method in Gree integration #101196
Allow setting hvac mode through set_temperature climate method in Gree integration #101196
Conversation
Hey there @cmroche, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
@@ -407,6 +414,58 @@ async def test_send_target_temperature( | |||
state.attributes.get(ATTR_CURRENT_TEMPERATURE) | |||
== fake_device.current_temperature | |||
) | |||
assert state.state == HVAC_MODES.get(fake_device.mode) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This adds a check that hvac mode didn't change in existing test for setting the temperature, while the new test checks that both temperature and hvac mode got changed.
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
Co-authored-by: G Johansson <goran.johansson@shiftit.se>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @MislavMandaric 👍
Proposed change
Support for handling optional
hvac_mode
parameter when callingclimate.set_temperature
on Gree integration, as per the docs.Previously this parameter was ignored.
Type of change
Checklist
black --fast homeassistant tests
)To help with the load of incoming pull requests: