DOCS-2743 / 27 / MinIO/AIStor trademark legal compliance (master) - #4775
Merged
Conversation
…page Per code review: the plan's casing-fix constraint is unconditional across the whole page, not limited to the brief's enumerated edits. This occurrence (third paragraph) was found during self-review but left unfixed; review flagged it as required. Casing-only fix, no bold/TM since this is not the page's first mention.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…ypos Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Hugo's taxonomy pages (tags, plus this site's book/volume taxonomies) render each listed page's auto-summary, which can include a page's marked first mention without its footer attribution (the summary truncates before reaching it). Split trademark-notice into a partial (reusable from any template) plus a thin shortcode wrapper, and wire taxonomy.html to call the partial for pages flagged via a new trademarkMinio/trademarkAistor frontmatter param. Since this template is repo-local, not vendored theme code, this fix also covers the remaining 8 branches once ported there.
Contributor
Vale Style Guide Issues (Previous - Click to expand)Vale Style Guide Issues📊 Summary: 325 issues found in changed files - 32 errors, 9 warnings, 284 suggestions Changed files with issues: content/_archive/COREReleaseNotes/12.0/12.0U4.md (83 issues)
content/SCALE/Apps/AppsScreens.md (61 issues)
content/SCALE/GettingStarted/Migrate/MigratePrep.md (58 issues)
content/_archive/COREReleaseNotes/12.0/12.0U8.md (53 issues)
content/_archive/COREReleaseNotes/12.0/12.0U8.1.md (28 issues)
content/Solutions/Integrations/VeeamImmutability.md (17 issues)
content/References/Copyrights.md (9 issues)
content/Solutions/MinioCOREtoSCALE.md (8 issues)
content/Solutions/MigratingToTrueNAS.md (5 issues)
content/Solutions/_index.md (3 issues)
🔧 To fix: Review the issues above and update your documentation accordingly. |
Vale Style Guide Issues📊 Summary: 325 issues found in changed files - 32 errors, 9 warnings, 284 suggestions Changed files with issues: content/_archive/COREReleaseNotes/12.0/12.0U4.md (83 issues)
content/SCALE/Apps/AppsScreens.md (61 issues)
content/SCALE/GettingStarted/Migrate/MigratePrep.md (58 issues)
content/_archive/COREReleaseNotes/12.0/12.0U8.md (53 issues)
content/_archive/COREReleaseNotes/12.0/12.0U8.1.md (28 issues)
content/Solutions/Integrations/VeeamImmutability.md (17 issues)
content/References/Copyrights.md (9 issues)
content/Solutions/MinioCOREtoSCALE.md (8 issues)
content/Solutions/MigratingToTrueNAS.md (5 issues)
content/Solutions/_index.md (3 issues)
🔧 To fix: Review the issues above and update your documentation accordingly. |
micjohnson777
approved these changes
Sep 3, 2026
Contributor
|
This PR has been merged and conversations have been locked. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Same legal requirement as the apps-web repo work (DOCS-2743, already merged/open there in 2 PRs) — this is branch 1 of 9 for the
/documentationrepo.masterestablishes the pattern the other 8 release branches will reuse.trademark-noticeshortcode + backing partial (layouts/partials/trademark-notice.html,layouts/shortcodes/trademark-notice.html) as the single source of truth for the exact attribution wording — the shortcode fails the build loudly (viaerrorf) if called without validminio/aistorparams, so a legal notice can't silently go missing on a typo.Copyrights.mdtrademarks page.content/_archive/COREReleaseNotes/changelogs.{{< children >}}), and Hugo's built-in taxonomy/tag pages (layouts/_default/taxonomy.html, a repo-local template — this fix is reusable on all 8 remaining branches without modification, unlike the vendored-themechildrenshortcode).Test plan
hugo --gc --minifybuilds clean (0 errors)trademark-noticeshortcode's fail-loud error path independently re-tested live (bad/missing params correctly fail the build){{< children >}}-surfaced mention, then a taxonomy-page-surfaced mention)Known pre-existing issue (not introduced by this PR, not fixed here)
Final review surfaced an unrelated bug:
content/SCALE/GettingStarted/Migrate/_index.mddeclares aliases that collide with two real pages' own URLs, causing Hugo to non-deterministically pick either the real page or a redirect stub at the same output path across builds. Predates this branch; recommend a separate ticket.🤖 Generated with Claude Code