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

Remove some cascade removes #1445

Merged
merged 1 commit into from
Feb 9, 2025
Merged

Conversation

BentiGorlich
Copy link
Member

@BentiGorlich BentiGorlich commented Feb 9, 2025

  • Some entities have very weird and possibly dangerous cascade removes, meaning when the entity is removed every referenced entity with this cascade remove property will also be removed by doctrine

Fixes: #1444

- Some entities have very weird and possibly dangerous cascade removes, meaning when the entity is removed every referenced entity with this cascade remove property will also be removed by doctrine
@BentiGorlich BentiGorlich added bug Something isn't working backend Backend related issues and pull requests high priority critical issue or PR that impacts production labels Feb 9, 2025
@BentiGorlich BentiGorlich added this to the v1.8.0 milestone Feb 9, 2025
@BentiGorlich BentiGorlich requested a review from melroy89 February 9, 2025 22:07
@BentiGorlich BentiGorlich self-assigned this Feb 9, 2025
@BentiGorlich BentiGorlich merged commit 95338ff into main Feb 9, 2025
7 checks passed
@BentiGorlich BentiGorlich deleted the fix/wrong-doctrine-cascade-removes branch February 9, 2025 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Backend related issues and pull requests bug Something isn't working high priority critical issue or PR that impacts production
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Purge notifications will remove users!
2 participants