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

[Addressing] Enable orphan removal for zone members #14059

Merged
merged 1 commit into from Jun 8, 2022

Conversation

coldic3
Copy link
Contributor

@coldic3 coldic3 commented Jun 7, 2022

Q A
Branch? 1.11
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Related tickets #14041
License MIT

Without orphan removal, even if we delete the zone member, the corresponding province/country cannot be removed because orphans still exist in the database.

@coldic3 coldic3 requested a review from a team as a code owner June 7, 2022 07:36
@coldic3 coldic3 force-pushed the fix/zone-members-orphan-removal branch 2 times, most recently from dea3359 to 43f8b39 Compare June 7, 2022 07:41
@coldic3 coldic3 changed the base branch from master to 1.11 June 7, 2022 08:08
@coldic3 coldic3 force-pushed the fix/zone-members-orphan-removal branch from 43f8b39 to 4332225 Compare June 7, 2022 08:43
@GSadee GSadee added the Bug Confirmed bugs or bugfixes. label Jun 7, 2022
@coldic3 coldic3 force-pushed the fix/zone-members-orphan-removal branch from 4332225 to 47fef6e Compare June 7, 2022 13:43
@GSadee GSadee merged commit 0be62fe into Sylius:1.11 Jun 8, 2022
@GSadee
Copy link
Member

GSadee commented Jun 8, 2022

Thanks, Kevin! 🥇

@coldic3 coldic3 deleted the fix/zone-members-orphan-removal branch June 8, 2022 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Confirmed bugs or bugfixes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants