Skip to content

Clarify orphan deletion in related-data.md - #5500

Merged
AndriySvyryd merged 2 commits into
dotnet:mainfrom
devinbroussard:patch-1
Oct 28, 2026
Merged

AndriySvyryd merged 2 commits into
dotnet:mainfrom
devinbroussard:patch-1

Conversation

@devinbroussard

Copy link
Copy Markdown
Contributor

Fixes #5455

Clarifies the example in the 'Removing relationships' section. The example was correct, but didn't explain why removing the post from blog.Posts deletes it, which is the confusion raised in the issue. Also adds a link to the 'Severing a relationship' section of the Cascade Delete page, which covers this case.

Also removes a stray double space in one of the sentences above.

@devinbroussard
devinbroussard marked this pull request as ready for review October 24, 2026 23:59
@AndriySvyryd
AndriySvyryd requested a lite review from Copilot October 28, 2026 01:14

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No active review issues were identified in this focused documentation clarification, link update, and spacing fix.

No open issues 🎉

What changed in this PR

Clarifies why removing a required Post from Blog.Posts causes EF Core to delete the orphan during SaveChanges, and links to the detailed cascade-delete guidance.

Changes:

  • Explains orphan deletion through cascade delete by convention.
  • Links to the “Severing a relationship” section.
  • Removes a stray double space.
File Description
entity-framework/​core/​saving/​related-data.md Clarifies orphan deletion behavior and fixes spacing in the “Removing relationships” section.

Review effort: Lite

@AndriySvyryd AndriySvyryd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution!

@AndriySvyryd
AndriySvyryd merged commit 7334a8b into dotnet:main Oct 28, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

I think there is a inaccuracy in the example in the "Removing relationships" section.

3 participants