Skip to content

Conversation

AndyButland
Copy link
Contributor

📋 Description

Added further detail to breaking changes for 17 relating to models builder and Razor runtime compilation

✅ Contributor Checklist

I've followed the Umbraco Documentation Style Guide and can confirm that:

  • Code blocks are correctly formatted.
  • Sentences are short and clear (preferably under 25 words).
  • Passive voice and first-person language (“we”, “I”) are avoided.
  • Relevant pages are linked.
  • All links work and point to the correct resources.
  • Screenshots or diagrams are included if useful.
  • Any code examples or instructions have been tested.
  • Typos, broken links, and broken images are fixed.

Product & Version (if relevant)

CMS 17

Deadline (if relevant)

Can merge at any time but publish on October 15th.

@eshanrnh
Copy link
Contributor

Thanks for the PR, @AndyButland 🙌 We will review it soon.

Copy link
Contributor

@eshanrnh eshanrnh left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, @AndyButland 🙌. I have added two suggestions to improve the flow.

Copy link

The Link Checker found broken links in your PR.
Please review the following list:

Broken links found in: 17/umbraco-cms/fundamentals/setup/upgrading/version-specific/README.md

⚓ Anchor not found → 17/umbraco-cms/fundamentals/setup/upgrading/version-specific#umbraco-14 | Cannot find fragment

⚠ Unknown error → [522] http://issues.umbraco.org/issue/U4-6595 |#124; Rejected status code (this depends on your "accept" configuration): Unknown status code

⚠ Unknown error → [522] http://issues.umbraco.org/issue/U4-8031 |#124; Rejected status code (this depends on your "accept" configuration): Unknown status code

⚠ Unknown error → [522] http://issues.umbraco.org/issues/U4?q=Due+in+version%3A+7.6.0+Backwards+compatible%3F%3A+No+ |#124; Rejected status code (this depends on your "accept" configuration): Unknown status codeFull Github Actions output

@eshanrnh eshanrnh merged commit 89ea75c into main Oct 13, 2025
18 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants