Skip to content

fix: change description that subscriptionExpireTime is optional#120

Merged
akoshunyadi merged 2 commits intocamaraproject:mainfrom
maxl2287:fix/subscriptionExpireTime-optional
Mar 13, 2024
Merged

fix: change description that subscriptionExpireTime is optional#120
akoshunyadi merged 2 commits intocamaraproject:mainfrom
maxl2287:fix/subscriptionExpireTime-optional

Conversation

@maxl2287
Copy link
Contributor

@maxl2287 maxl2287 commented Mar 5, 2024

What type of PR is this?

Add one of the following kinds:

  • correction

What this PR does / why we need it:

Correct the description for "subscription-ends" that this event will be send, when subscription expire time is optionally set by the requester.

Which issue(s) this PR fixes:

Fixes #116

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/

@maxl2287 maxl2287 requested a review from bigludo7 March 6, 2024 08:28
@akoshunyadi akoshunyadi mentioned this pull request Mar 6, 2024
bigludo7
bigludo7 previously approved these changes Mar 6, 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

@akoshunyadi akoshunyadi merged commit bec8015 into camaraproject:main Mar 13, 2024
@maxl2287 maxl2287 deleted the fix/subscriptionExpireTime-optional branch March 13, 2024 10:29
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.

subscriptionExpireTime is declared as "required" in the subscription-ends - note, but is optional currently

4 participants