Skip to content

Additional details on the media path scheme - #8594

Merged
sofietoft merged 2 commits into
mainfrom
cms/media-path-scheme-details
Oct 6, 2026
Merged

sofietoft merged 2 commits into
mainfrom
cms/media-path-scheme-details

Conversation

@AndyButland

Copy link
Copy Markdown
Contributor

📋 Description

Documents additional details on the media path scheme on how and when to use a custom media path scheme.

📎 Related Issues (if applicable)

https://github.com/umbraco/Umbraco-CMS/security/advisories/GHSA-7pj9-jmp9-p86q

✅ 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 and 18

Deadline (if relevant)

Anytime

@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown

Thanks for opening this PR! 🎉

Quick question: Does this PR include any AI-generated content?

AI tools are absolutely welcome here! We only ask that you follow our AI contribution guidelines to help keep the documentation consistent and trustworthy.

Drop a quick comment to let us know!

@AndyButland AndyButland added category/umbraco-cms ai-generated This PR was written by or with the assistance of AI. labels Oct 6, 2026

@sofietoft sofietoft left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a couple of minor things.

Should we mention in what version the new schemes are added?
Also, my review highligthed that the advisory references Umbraco 13 docs - is that intentional?

Co-authored-by: sofietoft <stk@umbraco.com>
@AndyButland

Copy link
Copy Markdown
Contributor Author

Thanks @sofietoft, I've accepted the suggestions.

Should we mention in what version the new schemes are added?

No, I don't think that's necessary. The two schemes listed have been there for a long time, I just took the time to document them whilst I was updating this. The only thing that's come in the latest release is one of the methods on the interface, and I don't think that's worth documenting with a patch version. In most cases we document what's in the latest version for the major that the docs are for.

Also, my review highlighted that the advisory references Umbraco 13 docs - is that intentional?

No, and that's a good catch - thanks. We should be linking to the 17 or 18 docs. I've fixed that now in the advisory.

@sofietoft

Copy link
Copy Markdown
Contributor

Super!
I'll get this merged then 💪

@sofietoft
sofietoft merged commit b3d4496 into main Oct 6, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-generated This PR was written by or with the assistance of AI. category/umbraco-cms

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants