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
HomematicIP fix cover direction #20901
HomematicIP fix cover direction #20901
Conversation
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.
Thank you for this fix! Just a couple suggestions that will greatly improve maintainability!
Can be merged once build completes |
In which version did you merged it? Unfortunately in 0.89dev it has still the wrong direction. |
Have installed the official 0.89.1 release - The problem is not fixed yet in the release as far as I can see. |
Please open an issue if you suspect a bug. If you need help please use our help channels: Merged PRs should not be used for support or bug reports. Thanks! |
Description:
Fix for the inverted cover direction.
**Related issue (if applicable): see comments: #19794 (comment)
Checklist:
tox
. Your PR cannot be merged unless tests passIf the code communicates with devices, web services, or third-party tools:
REQUIREMENTS
variable (example).requirements_all.txt
by runningscript/gen_requirements_all.py
..coveragerc
.