camaraproject / DeviceLocation

Repository to describe, develop, document and test the DeviceLocation API family
Apache License 2.0
20 stars 31 forks source link

Add handling when the requested geofence `area` cannot be covered #224

Open maxl2287 opened 4 days ago

maxl2287 commented 4 days ago

What type of PR is this?

Add one of the following kinds:

What this PR does / why we need it:

Adds handling when the provided subscription area cannot be covered or is too small to receive any notifications.

Which issue(s) this PR fixes:

Fixes #198

maxl2287 commented 4 days ago

@bigludo7 do you align with the TerminationReason?

bigludo7 commented 4 days ago

@bigludo7 do you align with the TerminationReason?

I've raised a generic issue in commonalities (243). Here we have a specific one. I'm not sure about commonalities outcome so probably we can keep this one open to get team feedback, and in // check if we have a global direction from commonalities.

maxl2287 commented 4 days ago

In DeviceStatus we have a label waiting for commonalities. Maybe we can also use such label here 🤔