Update the subscription models to align on CAMARA commonalities#170
Conversation
|
The base url should be with apiName, so 'device-' is missing.. (I can't comment there directly) |
code/API_definitions/device-reachability-status-subscriptions.yaml
Outdated
Show resolved
Hide resolved
code/API_definitions/device-reachability-status-subscriptions.yaml
Outdated
Show resolved
Hide resolved
code/API_definitions/device-reachability-status-subscriptions.yaml
Outdated
Show resolved
Hide resolved
bigludo7
left a comment
There was a problem hiding this comment.
LGTM
Nice job Max ! thanks
|
As per today discussion over subscription response, would apprecaite if we can include examples in specification that pointed in discussion |
|
@maxl2287 I have a second thought about making the device identifier optional in the subscription. But, when the developer check the subscription 10 days later, in the GET /subscriptions response better to have this device identifier present because without it impossible to know what device is targeted.... So... either 1/ we keep the device mandatory or 2/we make it optionnal for the POST request but mandatory in the GET response I prefer 1/ but perhaps we can have other solution. |
@bigludo7 I aggree also on letting it mandatory here. |
@sachinvodafone There should not be a mix of |
I suggest we do this way - let keep it mandatory. If in future we shift to optional this is not a breaking change. |
@maxl2287 I understand your point that it will only include those subscriptions that have already been created. Thank you. |
@sachinvodafone yes, but to be more precise: |
Suppose if we have only one subscription and that is asynchronous which still not completed, in this case ,we will get "ACTIVATION_REQUESTED" if user try for "GET /Subscriptions" . If Yes, then I am having another question. |
You will receive something like: [
{
"sink": "https://endpoint.example.com/sink",
"sinkCredential": {},
"types": [
"string"
],
"config": {
"subscriptionDetail": {
"device": {
"phoneNumber": "+123456789",
"networkAccessIdentifier": "123456789@domain.com",
"ipv4Address": {
"publicAddress": "84.125.93.10",
"publicPort": 59765
},
"ipv6Address": "2001:db8:85a3:8d3:1319:8a2e:370:7344"
}
},
"subscriptionExpireTime": "2023-01-17T13:18:23.682Z",
"subscriptionMaxEvents": 5,
"initialEvent": true
},
"id": "550e8400-e29b-41d4-a716-446655440000",
"startsAt": "2024-07-04T11:44:36.664Z",
"expiresAt": "2024-07-04T11:44:36.664Z",
"status": "ACTIVATION_REQUESTED"
}
] |
I have updated the scenario in discussion board here |
|
@bigludo7 can we do here a final review? |
Thanks @maxl2287 - Look good for me. Let's get final review from @sachinvodafone and @akoshunyadi and we're good to go. |
akoshunyadi
left a comment
There was a problem hiding this comment.
The server url seems to miss the device- prefix for roaming
|
@sachinvodafone it is ok for you now? Looking for you thumb up to merge :) |
…e/update-subscription-models # Conflicts: # code/API_definitions/device-reachability-status-subscriptions.yaml # code/API_definitions/device-roaming-status-subscriptions.yaml
What type of PR is this?
Add one of the following kinds:
What this PR does / why we need it:
Update of the subscription-models based on:
Which issue(s) this PR fixes:
Fixes #162