Skip to content

Conversation

@AndyButland
Copy link
Contributor

πŸ“‹ Description

This PR updates the code sample and instructions for the "Creating a Block Grid programmatically" on the "Block Grid" page

πŸ“Ž Related Issues (if applicable)

#7599

βœ… 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 16 and 17

Deadline (if relevant)

Can be reviewed and published at any time.

@eshanrnh
Copy link
Contributor

@AndyButland is this ready for internal or docs review? Asking cz the task is in internal community review status 😊

@AndyButland
Copy link
Contributor Author

You can review this one please @eshanrnh. It's an update to part of a page that HQ were asked to look into, where there rest of the page is being handled by a community team member. So it won't close the linked issue, rather is just related to it.

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 πŸ™Œ The PR looks good.

I have added suggestions to start the steps/list with an action word and cleaned up the long, messy image URLs. I'll take care of those.

@github-actions
Copy link

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

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

⚠ Unknown error β†’ [500] https://our.umbraco.com/download/releases/1300 |#124; Rejected status code (this depends on your "accept" configuration): Internal Server Error

@eshanrnh eshanrnh merged commit 9c49d92 into main Nov 12, 2025
40 of 48 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