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

fix(binary_sensor): changes async_write_ha_state method to schedule_update_ha_state in is_on() #351

Merged
merged 1 commit into from
May 4, 2024

Conversation

adutchak
Copy link
Contributor

@adutchak adutchak commented May 2, 2024

This fixes the bellow issue in HA 2024.5.0.

Failed to handle event, error: Detected that custom integration 'dahua' calls async_write_ha_state from a thread at custom_components/dahua/__init__.py, line 397: listener(). Please report it to the author of the 'dahua' custom integration., Line: 125
Failed to handle event, error: Detected that custom integration 'dahua' calls async_write_ha_state from a thread at custom_components/dahua/__init__.py, line 414: listener(). Please report it to the author of the 'dahua' custom integration., Line: 125

@adutchak adutchak force-pushed the fix-binary-sensors-async-write branch from 11fc562 to 7be2f0e Compare May 2, 2024 11:25
@adutchak adutchak changed the title feat(binary_sensor): changes async_write_ha_state method to schedule_update_ha_state in is_on() fix(binary_sensor): changes async_write_ha_state method to schedule_update_ha_state in is_on() May 2, 2024
@TECH7Fox
Copy link

TECH7Fox commented May 4, 2024

I'm also getting this error, and my entities are no longer updating. Would love to get this fixed.

@rroller
Copy link
Owner

rroller commented May 4, 2024

Sorry, I've been pretty busy and just now getting to this.

@rroller rroller merged commit 645cbf6 into rroller:main May 4, 2024
1 of 3 checks passed
@rroller
Copy link
Owner

rroller commented May 4, 2024

Released in 0.9.52
https://github.com/rroller/dahua/releases/tag/0.9.52

@TECH7Fox
Copy link

TECH7Fox commented May 4, 2024

Thank you! 🙏

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.

3 participants