-
Notifications
You must be signed in to change notification settings - Fork 570
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 async_dispatcher_send #1762
base: master
Are you sure you want to change the base?
fix async_dispatcher_send #1762
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.
I have checked this change in a LocalTuya implementation with 16 local devices and it resoves the issue. No regression found as yet.
fixes #1751 |
hey @fabio-garavini @orgrazm was wondering what else is needed to get this pr merged in? |
Also wanted to add, i tested this change with my 10 devices and it seems to have fixed the issue for me as well |
Unfortunately there is nothing more I can do. I don't have permissions/experience/knowledge to undertake further actions on this PR. In the end I have moved to a fork of this integration which has resolved the majority of my issues. Rospogrigio's life has taken over (which is fair) and I suspect this version will sit still for a bit until someone with relevant skills and time is able to take the lead - 1K issues and 100PRs will take a lot of effort and time to clear. |
Should resolve #1761
I've tested on my system and works, but I don't have a lot of devices to test it with (only 2 smart plugs)