Skip to content

Add termination reason SUBSCRIPTION_DELETED when subscription deleted by the user#121

Merged
bigludo7 merged 4 commits intocamaraproject:mainfrom
maxl2287:feature/cloud-event-when-delete-subscription
Jul 12, 2024
Merged

Add termination reason SUBSCRIPTION_DELETED when subscription deleted by the user#121
bigludo7 merged 4 commits intocamaraproject:mainfrom
maxl2287:feature/cloud-event-when-delete-subscription

Conversation

@maxl2287
Copy link
Contributor

@maxl2287 maxl2287 commented Mar 5, 2024

What type of PR is this?

Add one of the following kinds:

  • enhancement/feature

What this PR does / why we need it:

Adds a new termination-reason for "subscription-ends" - CloudEvent, when the user deletes the subscriptions via
DELETE /subscriptions/{subscriptionId}

Which issue(s) this PR fixes:

Fixes #117

@akoshunyadi
Copy link
Collaborator

akoshunyadi commented Mar 6, 2024

We should wait until the problem is solved in commonalities, we don't want to have proprietary solutions in each API

@akoshunyadi
Copy link
Collaborator

The discussion about subscription lifecycle management in commonalities: camaraproject/Commonalities#153

@maxl2287
Copy link
Contributor Author

waiting for megalinter fixes which are already done by #146

@maxl2287 maxl2287 changed the title feat: add termination reason when subscription deleted add termination reason when subscription deleted Jun 11, 2024
@maxl2287 maxl2287 changed the title add termination reason when subscription deleted Add termination reason SUBSCRIPTION_DELETED when subscription deleted Jun 11, 2024
@maxl2287 maxl2287 changed the title Add termination reason SUBSCRIPTION_DELETED when subscription deleted Add termination reason SUBSCRIPTION_DELETED when subscription deleted by the user Jun 11, 2024
@maxl2287 maxl2287 force-pushed the feature/cloud-event-when-delete-subscription branch from c307c75 to b20db75 Compare June 21, 2024 11:11
@maxl2287 maxl2287 self-assigned this Jun 21, 2024
@maxl2287
Copy link
Contributor Author

@camaraproject/device-status_maintainers
This can be reviewed and possibly merged after camaraproject/Commonalities#238 is merged

@maxl2287 maxl2287 requested a review from akoshunyadi June 24, 2024 07:36
akoshunyadi
akoshunyadi previously approved these changes Jun 24, 2024
@maxl2287 maxl2287 added enhancement New feature or request and removed waiting for commonalities labels Jun 25, 2024
@maxl2287
Copy link
Contributor Author

The related Communality issue is closed.
So we are ready to merge this one here.

bigludo7
bigludo7 previously approved these changes Jun 30, 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 Jul 4, 2024
…e/cloud-event-when-delete-subscription

# Conflicts:
#	code/API_definitions/device-reachability-status-subscriptions.yaml
#	code/API_definitions/device-roaming-status-subscriptions.yaml
@maxl2287 maxl2287 dismissed stale reviews from sachinvodafone, bigludo7, and akoshunyadi via b9a48b3 July 11, 2024 20:24
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/

@bigludo7 bigludo7 merged commit f7cda98 into camaraproject:main Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Fall24

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Send a CloudEvent when DELETE /subscriptions/{subscriptionId}

4 participants