camaraproject / DeviceLocation

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

Update location-retrieval.yaml #100

Closed bigludo7 closed 10 months ago

bigludo7 commented 10 months ago

Introduce Polygon

What type of PR is this?

Add one of the following kinds:

What this PR does / why we need it:

Allow to provided polygone-based location in the response.

Which issue(s) this PR fixes:

Fixes #94

Special notes for reviewers:

2 questions:

Changelog input

 release-note
- add polygon as area potentially provided in the response

Additional documentation

This section can be blank.

docs
alpaycetin74 commented 10 months ago

Re: "From API customer perspective, the customer doesn't know if a circle or a polygon will be responded. Not sure this is an issue but need to be considered in particular if we extend this liste of area." It may be helpful to indicate in the API definition that the retrieved shape depends on the network conditions at the subscriber's location and any of the supported shapes could be received. The consumer should be able to process any supported shape for each request.

jlurien commented 10 months ago

@bigludo7 please take a look to my comments, thanks for the proposal

bigludo7 commented 10 months ago

For your consideration, an alternate definition for polygon

Yes, this is way better ! thanks !!

nickvenezia commented 10 months ago

Has anyone addressed the privacy issues for opt-out or local privacy laws are met?

Would this be a new PR?

Where would I post?

bigludo7 commented 10 months ago

@nickvenezia For the questions/comments related to privacy did you checked https://github.com/camaraproject/IdentityAndConsentManagement project ? As this is a transverse topic for a lot of API this topic is tackled there.