Additional details on the media path scheme - #8594
Conversation
|
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! |
sofietoft
left a comment
There was a problem hiding this comment.
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>
|
Thanks @sofietoft, I've accepted the suggestions.
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.
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. |
|
Super! |
📋 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:
Product & Version (if relevant)
CMS 17 and 18
Deadline (if relevant)
Anytime