Skip to content

Conversation

@sofietoft
Copy link
Contributor

📋 Description

After more upgrade instructions have been added, I've restructure the section a tiny bit, and added more links to make it easier to navigate.

Deadline (if relevant)

ASAP.

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.

Looks good, @sofietoft 🙌 I have left a couple of comments here and there. Nothing major. Also, can you add the new article to the Summary.md file?

@sofietoft
Copy link
Contributor Author

Thanks for the review @eshanrnh 🙏
I've added the unattended article to the SUMMARY file.

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.

Found a couple of links that need to be fixed in the Upgrade unattended article and some other minor formatting stuff.


Are you running a load-balanced setup with multiple servers and environments?

Check out the section about [Unattended upgrades in a load-balanced setup](./#unattended-upgrades-in-a-load-balanced-setup).
Copy link
Contributor

Choose a reason for hiding this comment

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

This link takes to the Upgrade your project article. It needs to be fixed.


Follow the steps outlined below to use unattended upgrades in a load-balanced setup.

1. [Upgrade Umbraco via NuGet](./#upgrade-to-a-new-major).
Copy link
Contributor

Choose a reason for hiding this comment

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

This link needs to be fixed.

sofietoft and others added 3 commits August 21, 2025 09:42
Co-authored-by: Esha Noronha <82437098+eshanrnh@users.noreply.github.com>
@sofietoft
Copy link
Contributor Author

That should resolve all issues 🤞
Could you give a look again @eshanrnh ?

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.

Looks perfect 🙌

@sofietoft sofietoft merged commit 87d2292 into main Aug 21, 2025
3 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.

3 participants