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

[ecobee] Fix setting vacation hold temperatures and ignore unrealistic actual temperature #18113

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

mhilbush
Copy link
Contributor

Fixes #18110 and #15042

Signed-off-by: Mark Hilbush mark@hilbush.com

Signed-off-by: Mark Hilbush <mark@hilbush.com>
Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fixes!

@jlaur jlaur added the bug An unexpected problem or unintended behavior of an add-on label Jan 16, 2025
@mhilbush
Copy link
Contributor Author

Do you think this commit can be cherry picked for 4.x?

@jlaur jlaur changed the title [ecobee] Fix open issues [ecobee] Fix setting vacation temperatures and fix updating actual temperature to -270°F Jan 16, 2025
@jlaur
Copy link
Contributor

jlaur commented Jan 16, 2025

Do you think this commit can be cherry picked for 4.x?

Yes, sure, it looks solid and quite low risk. 👍 Please review the new PR title as I was struggling trying to cram description of the two fixes into one sentence. 😉

@jlaur jlaur changed the title [ecobee] Fix setting vacation temperatures and fix updating actual temperature to -270°F [ecobee] Fix setting vacation temperatures and fix actual temperature being updated to -270°F Jan 16, 2025
@mhilbush mhilbush changed the title [ecobee] Fix setting vacation temperatures and fix actual temperature being updated to -270°F [ecobee] Fix setting vacation hold temperatures and ignore unrealistic actual temperature Jan 16, 2025
@mhilbush
Copy link
Contributor Author

Please review the new PR title as I was struggling trying to cram description of the two fixes into one sentence.

Haha, yeah, I was struggling with that too.

I gave it a shot.

@jlaur jlaur merged commit dfeac54 into openhab:main Jan 16, 2025
2 checks passed
@jlaur jlaur added this to the 5.0 milestone Jan 16, 2025
jlaur pushed a commit that referenced this pull request Jan 16, 2025
Signed-off-by: Mark Hilbush <mark@hilbush.com>
@jlaur jlaur added the patch A PR that has been cherry-picked to a patch release branch label Jan 16, 2025
chingon007 pushed a commit to chingon007/openhab-addons that referenced this pull request Jan 22, 2025
Signed-off-by: Mark Hilbush <mark@hilbush.com>
Signed-off-by: chingon007 <tron81@gmx.de>
chilobo pushed a commit to chilobo/openhab-addons that referenced this pull request Feb 10, 2025
Signed-off-by: Mark Hilbush <mark@hilbush.com>
Signed-off-by: Christian Koch <78686276+chilobo@users.noreply.github.com>
PascalTurbo pushed a commit to PascalTurbo/openhab-addons that referenced this pull request Feb 11, 2025
Signed-off-by: Mark Hilbush <mark@hilbush.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An unexpected problem or unintended behavior of an add-on patch A PR that has been cherry-picked to a patch release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ecobee] Vacation creation via JS rule ignores temp setpoints
2 participants