camaraproject / DeviceStatus

Repository to describe, develop, document and test the Device Status API family
Apache License 2.0
11 stars 33 forks source link

Addition of "lastChecked" Field #145

Closed sachinvodafone closed 5 months ago

sachinvodafone commented 5 months ago

What type of PR is this? Add one of the following kinds:

correction

What this PR does / why we need it: The CAMARA version of the current DeviceStatus API lacks a crucial "lastChecked" field, resulting in inconsistent behavior across vendors. This absence poses challenges for application developers in determining the freshness of roaming status/connectivity data.

Which issue(s) this PR fixes: Fixes https://github.com/camaraproject/DeviceStatus/issues/110

fernandopradocabrillo commented 5 months ago

Hi @sachinvodafone Do you really think that we need to have this attribute mandatory in the response? I was expecting to have this one as optional. Not a blocker for me.

Hi, my understanding was the same. I also think that the property should be optional and, in addition to that, the descriptions should be updated accordingly. By being optional we can define its behaviour as: if the property is not included in the response then the information is fresh. If it is included, it is not.

The desciption of the endpoint specifies:

Now it may not be the current so the description should reflect that.

sachinvodafone commented 5 months ago

Hi @sachinvodafone Do you really think that we need to have this attribute mandatory in the response? I was expecting to have this one as optional. Not a blocker for me.

Noted and will update accordingly