Skip to content

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

Merged
akoshunyadi merged 2 commits intocamaraproject:mainfrom
maxl2287:fix/add-missing-components
Sep 19, 2024
Merged

Add missing protocol-components for device-roaming-status-subscriptions#208
akoshunyadi merged 2 commits intocamaraproject:mainfrom
maxl2287:fix/add-missing-components

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
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