-
-
Notifications
You must be signed in to change notification settings - Fork 121
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
Fixed and added entity toggles #607
Conversation
I can't tell you how excited I am to get some of these changes added!!!!! The floodlight pro updates are something I've been dying for since Wyze launched them. THANK YOU KyGuy2002 for figuring these out! This is EXTREMELY appreciated! I hope we get a new update with these in them soon! |
@KyGuy2002 this is awesome, thank you for all this work! I’ll take a look and test the devices that I have soon. |
Thanks both of you! I am happy to help out :D |
I can toggle the Wyze notifications in the Home Assistant app and it does not turn on the alarm notifications in the Wyze app. I can do a reverse way turning the Wyze notifications on or off in the Wyze app and it does not affect the way the toggle is in the Home Assistant app either, I cannot automatically toggle the Wyze notifications through Home Assistant since about March or April. |
I'm not a developer and am relatively git-ignorant.... is there a way that I can use the changes in this PR and it's dependent so that I can take advantage of these additional functions? I've got a few of the Floodlight v2 cams and would really like to be able to control the lights from HA. Thanks! |
Unfortunately it's not super simple to do so. You would need to fork this repo, make some changes to directly reference the updated commits in the dependent library, and then uninstall the current wyze integration and add this one, which may cause you to loose your devices config (not sure). I think these PRs should be merged so users can take advantage of all the changes, and we can work on the other issues mentioned here without holding up these changes. |
Thanks for the quick response! I was able to copy the changes in the changed files in this PR and update those files in the HA custom_components/wyzeapi folder, but when I tried to do the same with the changed files in 91, I found that I couldn't locate those files in my installation. I've managed also to fork this repo, so if there are steps outlined somewhere on how to make those changes to directly reference the updated commits in 91, I could give that a shot. Thanks again for your responsiveness! |
@terryhonn you can download the files for the dependency update (91) and just place the new |
Thanks for all your hard work and getting this working. I have never downloaded the dependency files. If you can point me or send me a link on how to get to them I’ll download them and get them put in and test it. Thanks again for all your help. |
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.
Tested with the few devices I have. No issues. 👍
Will let @SecKatie take a final look before merging.
I’m just checking but is the global notification toggle in Home Assistant fixed with this version? So if I toggle it on or off, it will either put a line through the bell in the app or take it away? |
@mikefrantz no the switch is still broken. It's unclear at this point if its fixable. |
Reword comment to show only V1 Campan not having a siren Signed-off-by: Brian Rogers <brg468@hotmail.com>
759f2f1
to
e12ef51
Compare
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.
Looks good now. Let's get it in.
I tested and fixed many of the control entities in this integration. I also added some that were unavailable and removed some that are not applicable to the device they are on.
Note: These changes depend on changes made in SecKatie/wyzeapy#91
Note: I squashed all my commits into one. This is the first major HA contribution I made so I wanted to clean up all the commits where I was troubleshooting the HACS integration installation stuff.
Changes
Not Working (Proprietary/Tutk Communication Protocol)
Could Not Get Working/Needs Testing