Skip to content

Updated in-line documentation for Roaming to clarify that country information can optionally be returned in the response.#127

Merged
akoshunyadi merged 1 commit intocamaraproject:mainfrom
trehman-gsma:trehman/roaming-documentation-update-issue-124
Mar 27, 2024
Merged

Updated in-line documentation for Roaming to clarify that country information can optionally be returned in the response.#127
akoshunyadi merged 1 commit intocamaraproject:mainfrom
trehman-gsma:trehman/roaming-documentation-update-issue-124

Conversation

@trehman-gsma
Copy link
Contributor

@trehman-gsma trehman-gsma commented Mar 13, 2024

What type of PR is this?

  • documentation

What this PR does / why we need it:

Small update of in-line documentation for Roaming to clarify that country information can optionally be returned in the response. Previous documentation implied that country information was mandatory when roaming is true.

Which issue(s) this PR fixes:

Fixes #124

Special notes for reviewers:

As discussed in DeviceStatus meeting on 13-03-24. It was agreed that country information is optional in cases of roaming=true.

Changelog input

 release-note
 Updated in-line documentation for Roaming to clarify that country information in response is optional.

Updating in-line documentation for Roaming to clarify that country information can optionally be returned in the response.
Copy link
Collaborator

@bigludo7 bigludo7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bigludo7
Copy link
Collaborator

@sachinvodafone if you can review - this is an easy one ;)

@akoshunyadi akoshunyadi merged commit b9b08c4 into camaraproject:main Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When roaming status is true, is it mandatory to return both countryCode and countryName?

4 participants