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

Update all async_schedule_update_ha_state calls to their non-async counterpart #75

Merged
merged 5 commits into from
May 4, 2024

Conversation

optiluca
Copy link
Contributor

@optiluca optiluca commented May 4, 2024

This fixes the integration in Home Assistant 2024.5. Without it, these kinds of errors are thrown, and state is not correctly communicated to HA. https://community.home-assistant.io/t/how-to-fix-custom-integration-brematic-calls-async-write-ha-state/722238/3

@optiluca
Copy link
Contributor Author

optiluca commented May 4, 2024

Still having the issue whereby more commits are shown than is correct... But there only is 1 "real" commit in this PR.

@optiluca optiluca changed the base branch from master to 42-add-vedo-sensors May 4, 2024 06:51
@optiluca optiluca changed the base branch from 42-add-vedo-sensors to master May 4, 2024 06:51
@gicamm gicamm merged commit 3fa5796 into gicamm:master May 4, 2024
@gicamm
Copy link
Owner

gicamm commented May 4, 2024

Thank you Luca :)

gicamm pushed a commit that referenced this pull request May 4, 2024
…unterpart (#75)

* Inverted cover position to match home assistant's definition

* Update all async_schedule_update_ha_state calls to their non-async counterpart.
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.

None yet

2 participants