Skip to content

Conversation

@Musealali
Copy link
Contributor

Description

What did you add/update/change?

Type of suggestion

  • Typo/grammar fix
  • Updated outdated content
  • New content
  • Updates related to a new version
  • Other

Product & version (if relevant)

Deadline (if relevant)

When should the content be published?

@sofietoft
Copy link
Contributor

Hi @Musealali !

Thanks for the PR.
Please remember to name your PRs or the commits.
Should we need to go back in time to find something related to this, it is a big help that there's a name indicating what was changed 🙏

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, @Musealali 💪 I've made a small suggestion to change the Note type to Warning and add inline code tags for /umbraco.

Musealali and others added 2 commits April 4, 2025 10:52
Co-authored-by: Esha Noronha <82437098+eshanrnh@users.noreply.github.com>
@Musealali
Copy link
Contributor Author

@sofietoft

Sorry - Will try to remember next time.

@Musealali
Copy link
Contributor Author

@eshanrnh

I actually erased that warning / info thing - just talked with the team and they really like a limitations section just like in the database backups docs - so I've gone with that flow instead 😄

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 to me, @Musealali 🙌

@eshanrnh eshanrnh changed the title Update README.md Added limitations section in Manage Hostnames article Apr 4, 2025
@eshanrnh eshanrnh merged commit 152b5ee into main Apr 4, 2025
7 of 9 checks passed
@sofietoft sofietoft deleted the Musealali-patch-2 branch April 28, 2025 07:14
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.

4 participants