Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing protocol-components for device-roaming-status-subscriptions #208

Merged
merged 2 commits into from
Sep 19, 2024

Conversation

maxl2287
Copy link
Contributor

What type of PR is this?

Add one of the following kinds:

  • bug

What this PR does / why we need it:

This PR adds the missing components, which were referenced in the protocol-discriminator

Which issue(s) this PR fixes:

Fixes #207

@maxl2287 maxl2287 self-assigned this Sep 16, 2024
@maxl2287 maxl2287 added the bug Something isn't working label Sep 16, 2024
bigludo7
bigludo7 previously approved these changes Sep 17, 2024
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

sachinvodafone
sachinvodafone previously approved these changes Sep 17, 2024
Copy link
Collaborator

@akoshunyadi akoshunyadi left a comment

Choose a reason for hiding this comment

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

Sorry, just forgot about it. The first change after 0.6.0 should set the version back to wip.

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

@akoshunyadi
Copy link
Collaborator

@maxl2287 now you set the version for all of the APIs to wip. I think generally it is not necessary, because it is possible that an API remains unchanged in the upcoming version, so only the changed API should be set to wip and later for the release to the new version. Of course we could set the previous version again, if there was no change... Anyway, let's keep it for now.

@akoshunyadi akoshunyadi merged commit 2ebbb49 into camaraproject:main Sep 19, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Device-Roaming-Subscription]: Protocol components are missing in specification
5 participants