Skip to content

Rename subscription-ends event to subscription-ended#28

Merged
akoshunyadi merged 4 commits intocamaraproject:mainfrom
eric-murray:eric-murray-patch-4
Jul 1, 2025
Merged

Rename subscription-ends event to subscription-ended#28
akoshunyadi merged 4 commits intocamaraproject:mainfrom
eric-murray:eric-murray-patch-4

Conversation

@eric-murray
Copy link
Contributor

What type of PR is this?

  • correction

What this PR does / why we need it:

Commonalities has renamed the subscription-ends event to subscription-ended. This PR renames the event where it is used.

Which issue(s) this PR fixes:

Fixes #25

Special notes for reviewers:

I've not added the subscription-started or subscription-updated events as these are optional and require discussion.

Changelog input

 release-note
 - Rename `subscription-ends` event to `subscription-ended`

Additional documentation

None

sachinvodafone
sachinvodafone previously approved these changes Jun 25, 2025
bigludo7
bigludo7 previously approved these changes Jun 29, 2025
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
Copy link
Contributor

There are still such names there: SUBSCRIPTION_ENDS, EventSubscriptionEnds, SubscriptionEnds
I suggest to rename those too.

@eric-murray eric-murray dismissed stale reviews from bigludo7 and sachinvodafone via 1685b3a June 30, 2025 12:50
@eric-murray
Copy link
Contributor Author

There are still such names there: SUBSCRIPTION_ENDS, EventSubscriptionEnds, SubscriptionEnds I suggest to rename those too.

OK, done.

@akoshunyadi akoshunyadi merged commit 52e3384 into camaraproject:main Jul 1, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename subscription-ends event to subscription-ended

4 participants