Skip to content

Adds some general information about considerations for upgrades #7288

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

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

Conversation

AndyButland
Copy link
Contributor

πŸ“‹ Description

Adds some general information about considerations for upgrades

βœ… 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)

Umbraco CMS

Deadline (if relevant)

Any time. Once approved can be copied to at least the Umbraco 13 pages, so we have it on the last LTS.

@sofietoft
Copy link
Contributor

Thanks for the PR @AndyButland !

While I think this is very valuable information, I'm concerned about adding it to an already very long article.
Perhaps we could add this information into the "Expandables", like the ones we use in the Breaking Changes section? That way the information is not adding too much to the length of the article, and it's an opt-in to learn more about it.

That's my two cents.
In the docs team, we have an upcoming project to look into how we structure these upgrade docs, and hopefully make them a little easier to navigate.

@AndyButland
Copy link
Contributor Author

I see the point @sofietoft but what I was aiming for here was a general introduction to upgrades that is worth reading before you dive into the detail. As I do see some confusion about what we are talking about when we discuss these. So rather than hiding it in an expandable section, if you feel this page is too long, I think I'd prefer moving what I've added to a new page and placing it before the existing one in the summary. Then you can dive straight into the detail if you know the background, but if not, you'll hopefully see this new content first.

Let's get @BoletteKern's view too though - as this I've put this together following a request she had having had some feedback on this topic.

@sofietoft
Copy link
Contributor

Yes, splitting up is also a good idea! We can always highlight the page with a link from the article, as something we recommend going through before initiating the upgrade process.

We'll wait for Bolette, before we make any changes for now πŸ™

Copy link
Contributor

@BoletteKern BoletteKern left a comment

Choose a reason for hiding this comment

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

Hi Andy!!
This gives a great overview of what users don't need to worry, and what they should consider and test when upgrading. I only have small suggestions, and mainly about lilnking to more information.

I think the "What is involved in an upgrade" and "Before you upgrade" should be in one article, and "upgrade to a new major", "upgrade to a new minor" and "Run an unattended upgrade" as subarticles.

Copy link
Contributor

Choose a reason for hiding this comment

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

Can we refer to where this communication is made, e.g., Release Notes and Announcements on GitHub, and Release Blog Post on umbraco.com?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I do have that actually - in the sentence:

Umbraco communicates about the breaking changes in release blog posts and on the documented version specific upgrade details.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think we use "obselete" more than we use "legacy". Perhaps rewrite to:

They are only removed in major versions and with plenty of notice under announcements on the Umbraco repository on GitHub. When a feature is deprecated, a minor version is released with the deprecation, and the feature is deprecated for the next major release.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Have rewritten this slightly - mainly to add reference to the announcements repository, which is good to call out I agree. On the wording though, I think legacy is right here when we talk about features, and it's the word we used when retiring "Nested Content" for example. Obsolete is more used for code APIs.

Copy link
Contributor

Choose a reason for hiding this comment

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

All good, if "legacy" is the right wording. I just noticed we use "deprecation" on this page https://umbraco.com/products/knowledge-center/versioning-and-release-cadence/

@AndyButland
Copy link
Contributor Author

I think the "What is involved in an upgrade" and "Before you upgrade" should be in one article, and "upgrade to a new major", "upgrade to a new minor" and "Run an unattended upgrade" as subarticles.

I've split the article now as per this suggestions. Please see what you both think.

@sofietoft
Copy link
Contributor

Changes looks good.

I think it makes sense to split it up even further like Bolette suggest. So, having the different types of upgrades in each their own article. One for Major upgrades, one for minor and finally, one for running upgrades unattended.

I also think we should mentioned about the version specific guides and notes, on the new landing page for this section.

I'll make these addition after we've merged this PR.

@sofietoft sofietoft requested a review from BoletteKern August 12, 2025 10:41
Copy link
Contributor

@BoletteKern BoletteKern left a comment

Choose a reason for hiding this comment

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

I think the changes look good, Sofie 🀩

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.

3 participants