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

Block Customer deletion if they have active Subscription or Entitlements #2262

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

GAlexIHU
Copy link
Contributor

Overview

As the name suggests

Notes for Reviewer

Fetching all this from the DB is not ideal but deleting customers is a fairly infrequent operation

@GAlexIHU GAlexIHU added the release-note/bug-fix Release note: Bug Fixes label Feb 13, 2025
@GAlexIHU GAlexIHU enabled auto-merge (squash) February 13, 2025 11:01
@GAlexIHU GAlexIHU force-pushed the customer-delete-with-sub branch from b8d6952 to 7d3b232 Compare February 26, 2025 11:27
@GAlexIHU GAlexIHU requested a review from a team as a code owner February 26, 2025 11:27
@turip
Copy link
Contributor

turip commented Feb 26, 2025

I have implemented a minimal validation support for this yesterday here: #2343

I would love if we use that here too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/bug-fix Release note: Bug Fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants