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

making the changes from PR 315 to the most recent release (0.9.59) to solve floodlight control #372

Merged
merged 1 commit into from
Jul 20, 2024

Conversation

skooj
Copy link
Contributor

@skooj skooj commented Jul 17, 2024

At the risk of pestering, I am going to make another PR, as it seems this issue continues to get overlooked and the previous pull request (by another user) was never merged.

I was experiencing the issue that was possibly solved with PR 315 for the ability to control the light on my ASH26-W floodlight. Using that fork I was able to get my floodlight to work. However, it was very much an older version and the doorbell button on my AD410 would not trigger in home assistant.

I simply took the changes made to that pull request, applied them to the main branch 0.9.59, and have been using this fork for roughly a week with no issue.

I do not have any other devices aside from my ASH26-W and my AD410, so I am not positive if these changes will break anything else.

@skooj skooj changed the title making the changes from PR 315 to the most recent release (0.9.59) making the changes from PR 315 to the most recent release (0.9.59) to solve floodlight control Jul 17, 2024
@rroller
Copy link
Owner

rroller commented Jul 20, 2024

At the risk of pestering

Not at all! Thank you, sorry I've been busy and haven't had a lot of time to look at this. Taking a look now!

@rroller rroller merged commit 7979a18 into rroller:main Jul 20, 2024
1 of 3 checks passed
@rroller
Copy link
Owner

rroller commented Jul 20, 2024

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

@trailiajohn
Copy link

Thank you, I can finally update from 0.9.49

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