search

camaraproject / DeviceLocation

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

add /subscription-endpoint for location-area #49

Closed maxl2287 closed 1 year ago

maxl2287 commented 1 year ago

This PR is a follow up PR to the closed #27 This PR is a reference to #18.

It needs to be updated when the discussion abo is finalized.

fyi @bigludo7 & @jlurien

bigludo7 commented 1 year ago

@maxl2287 We have been (at last) to close the notification topic --> https://github.com/camaraproject/WorkingGroups/blob/main/Commonalities/documentation/API-design-guidelines.md#12-subscription-notification--event We are now in position to craft the API with subscription/notification. Are you able to contribute this API improvement? Thanks

maxl2287 commented 1 year ago

@bigludo7 thanks for the hint! I will start with that asap!

bigludo7 commented 1 year ago

@maxl2287 Thanks !! This is great One point still to be discussed and I will trigger an issue to get team feedback: Do we add the subscription/notification in current API or do we want it in a separate API? As we can have distinct business case for one shot check vs subscription model perhaps having 2 separate API make sense.

akoshunyadi commented 1 year ago

As decided in issue 64 the endpoint should be implemented in a separate API

akoshunyadi commented 1 year ago

@maxl2287 there is a new PR, this one can be closed