Skip to content

Conversation

@eric-murray
Copy link
Contributor

Updated the description of LastStatusTime to clarify its meaning and potential limitations.

What type of PR is this?

  • correction

What this PR does / why we need it:

Current description refers to reachability rather than roaming status, is slightly ambiguous, and includes formatting recommendations no longer required by Commonalities

Which issue(s) this PR fixes:

Fixes #55

Special notes for reviewers:

None

Changelog input

 release-note
 - Clarify LastStatusTime schema description in YAML

Additional documentation

None

Updated the description of LastStatusTime to clarify its meaning and potential limitations.
Corrected a typo in the LastStatusTime description.
@rartych
Copy link

rartych commented Nov 21, 2025

Maybe this fix: camaraproject/Commonalities#530 ( domain.com to example.com) can be applied to:

NetworkAccessIdentifier:
description: A public identifier addressing a subscription in a mobile network. In 3GPP terminology, it corresponds to the GPSI formatted with the External Identifier ({Local Identifier}@{Domain Identifier}). Unlike the telephone number, the network access identifier is not subjected to portability ruling in force, and is individually managed by each operator.
type: string
example: "123456789@domain.com"

@eric-murray
Copy link
Contributor Author

Maybe this fix: camaraproject/Commonalities#530 ( domain.com to example.com) can be applied to:

I'll do this as a separate PR - we should try and maintain best practice of not fixing multiple issues in the same PR, and we have a lot of time until the next release

Copy link
Contributor

@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

@akoshunyadi akoshunyadi merged commit f69eecc into camaraproject:main Dec 19, 2025
2 checks passed
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.

Typo in LastStatusTime schema description

5 participants