-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
After update to v0.1.1 Doorbell shows as accessory not supported #18
Comments
Yeah I confirm from the last update the doorbell is shown as not supported but if you ring you will receive a notification. We have to make a choice between motion sensor or receive ringing events. |
Personally, ring events and video would be more important to me. |
+1 |
As I read the ring homebridge plugin (https://github.com/dgreif/ring/tree/master/homebridge#homebridge-ring) it adds the doorbell as a camera accessory, so it has motion and batteryservice and then has a doorbell programmable switch accessory. |
@schliemann I might be misunderstanding this so apologies in advance but what @PeterAlexander75 and myself mean by ring events is doorbell presses, not ring as in Ring cameras. |
I am aware. I am just linking to a plugin that solved this issue. |
ah I see - thanks |
I've pushed a new implementation of DoorBell. But it's still noisy in term of debug. |
Although it's not taking homebridge down, I am getting repeated logs showing:-
|
Yeah ! saw it forget to update the code for camera device should be good now. |
Updated and redeployed, Still seeing "This accessory is not currently supported by the Home app." Wired only Doorbell 2k - T8200 - Log and image of error attached. I got this error on homebridge boot
|
I've updated to v0.1.3 but I'm not seeing a doorbell. I just have the Homebase and a motion sensor. Is this as expected? |
It doesn't appear as dummy object but if you press the button you will receive a notification from apple home app. |
It should work the function has battery is herited from device class. it check if the device has battery if not we don't published any battery notification. Are you sure you are using the latest version of the plugin ? I've added more context on debug for the next version. It would be more easy for us to troubleshoot |
Hi guys, I have attempted to implement live streaming on the doorbell and maybe remove some of the problems. However, I cannot test it. Can someone try deploying it and see if it works? It's in a separate branch and not published to NPM. |
Missing one file 4 import { EufyCameraStreamingDelegate } from './new-streaming-delegate'; |
Hum not working for me : |
Oops my bad, fixed both of those typos. Had the wrong file name and for the snapshot request, didn't call the function correctly. Try again now with the latest commit. |
no thumbnail and no stream |
I know this is a dumb question, but my experience is you should always ask them, but you do have ffmpeg installed? |
Not a dumb question. I might be able to test later tonight. |
Not a dumb question at all ! I was on my test platform and didn't installed ffmpeg !!! mea culpa ! It's working ! |
Awesome |
Nice!! Yeah forgot to mention you would need to install that. If this is working, I'll merge the branch into master and update NPM. |
Should we use the Homebridge Camera FFmpeg plug-in or the native ffmpeg? |
@javierireyes I think the homebridge one but either probably would work. |
The text was updated successfully, but these errors were encountered: