search

fwestenberg / reolink_dev

Home Assistant Reolink addon
MIT License
550 stars 102 forks source link

request - Push notifications change #339

Open cokeman0 opened 3 years ago

cokeman0 commented 3 years ago

would be possible to have "Push notifications", split into 3

Push notifications - Any motion Push notifications - Person Push notifications - Vehicle

I have E1 outdoor, and 510WA, both can under push notification select the 3 different types, but I'd like to only tun on/off "Push notifications person"

Aturner4444 commented 3 years ago

I'd like to add to this, I'm not sure where you are located but it seems that there is a difference between the UK and US firmware on the cameras or NVR.

I am unable to turn push notifications on or off for an individual camera, underneath the setting for push notifications on the app it states this will disable notifications for all cameras connected to the NVR. However on the support articles on ReoLinks website this section of the app does not state this and instead has 2 options, one for the NVR and one for each camera.

I would like to be able to control this with more selectivity, but I realise this is probably down to ReoLink.

You and or others may already be aware of this, if not I can post some screenshots showing what I mean.

cokeman0 commented 3 years ago

Well no need to prove it, I (we) trust you :-)

I have no NVR, and i'm i Denmark, så might be UK software.

cpainchaud commented 3 years ago

well, it's 'complicated' as in fact it's a schedule and not a button (push notification for each time is defined by a time table). May be an option to enable/disable the whole timetable

Aturner4444 commented 3 years ago

So there is no individual camera push notification control, just for on and off?

As ReoLink seem to state that there is and isn't.

Yeah the schedule for the camera controls the detection types, but I suppose this could possibly also offer a solution.

cpainchaud commented 3 years ago

That would be available as a service which could be called rather than a button then, meaning you would have no state. That or we put state true if ALL THE TIME TABLE is set to true....

cokeman0 commented 3 years ago

In my use case, I will use distance from home, if away activate "push on person", and when home, disable it. The enable/disable will be setup in HA automation, so I will not have the need for the build in time table.

Aturner4444 commented 3 years ago

That doesn't add the control over the specific type of detection that is enabled, person vehicle or any motion.

It may work for me though, as I have no requirement to alter the motion detection types, at least in my current use.

I can't help but feel that this functionality must still be present to turn notifications on and off on a per camera basis as it did exist in the app from what I can see, I however cannot see which region / firmware it existed on.

This is @cokeman0's request so I don't want to hijack it for my own purposes, but both levels of control would be beneficial.

In this article:

https://support.reolink.com/hc/en-us/articles/360013378753-How-to-Enable-Push-Notifications-for-NVR-via-Reolink-App

It shows that individual camera control is available, however this is not the case for me. Do you think they altered this feature for some reason or its a regional thing?

Would it be possible using a webhook to find the state of the detection types that are enabled and alter them? I've seen this done with regex but HA is a completely new platform for me so you would be better positioned to tell me I'm way of track.

cpainchaud commented 3 years ago

I don't have a NVR so feel free to reverse-engineer the API, if you find the API call then I can make it happen.

On Tue, Oct 26, 2021, 20:02 Aturner4444 @.***> wrote:

That doesn't add the control over the specific type of detection that is enabled, person vehicle or any motion.

It may work for me though, as I have no requirement to alter the motion detection types, at least in my current use.

I can't help but feel that this functionality must still be present to turn notifications on and off on a per camera basis as it did exist in the app from what I can see, I however cannot see which region / firmware it existed on.

This is @cokeman0 https://github.com/cokeman0's request so I don't want to hijack it for my own purposes, but both levels of control would be beneficial.

In this article:

https://support.reolink.com/hc/en-us/articles/360013378753-How-to-Enable-Push-Notifications-for-NVR-via-Reolink-App

It shows that individual camera control is available, however this is not the case for me. Do you think they altered this feature for some reason or its a regional thing?

Would it be possible using a webhook to find the state of the detection types that are enabled and alter them? I've seen this done with regex but HA is a completely new platform for me so you would be better positioned to tell me I'm way of track.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/fwestenberg/reolink_dev/issues/339#issuecomment-952180335, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABTC5PRIUZKAZQG4743QVBDUI33K5ANCNFSM5GVN54FA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.