location-verification.yaml v0.2 Release Candidate

camaraproject / DeviceLocation

Repository to describe, develop, document and test the DeviceLocation API family

Apache License 2.0

21 stars 33 forks source link

location-verification.yaml v0.2 Release Candidate #104

Closed jlurien closed 8 months ago

jlurien commented 10 months ago

What type of PR is this?

correction
cleanup
documentation

What this PR does / why we need it:

Update documention in info.description with latest atributes included
Examples updated and enhanced
Document behaviour when maxAge cannot be satisfied
Removed required for lastLocationTime in response as there may be cases when there is no historical location data for the device
IPV6 address must identify a single address

Which issue(s) this PR fixes:

Fixes #102 , #112

alpaycetin74 commented 9 months ago

Is it helpful to extend the definition of UNKNOWN, to state it also covers the case where there is a location, but older than maxAge ? It could read: "when the network cannot locate the device within the specified maxAge, or cannot locate at all"

alpaycetin74 commented 9 months ago

Apologies, I think I got the PARTIAL one wrong. It should read: If the network's estimation of the device's location partially overlaps with the requested area, or it fully contains the requested area, the result is 'PARTIAL'. Thank you

jlurien commented 9 months ago

Apologies, I think I got the PARTIAL one wrong. It should read: If the network's estimation of the device's location partially overlaps with the requested area, or it fully contains the requested area, the result is 'PARTIAL'. Thank you

Corrected, no problem, I should have noticed

bigludo7 commented 9 months ago

@jlurien We have still to wait for IP v6 fix before to merge this one - correct?

jlurien commented 8 months ago

@jlurien We have still to wait for IP v6 fix before to merge this one - correct?

It is almost closed in QoD, so I hope we can apply it by tomorrow. We'll have to align the 3 specs

jlurien commented 8 months ago

A small proposal for lastLocationTime description.

I see it is already committed. Fine