-
Notifications
You must be signed in to change notification settings - Fork 945
Rename Docsy's render-heading hook and helper partial #2223
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename Docsy's render-heading hook and helper partial #2223
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR renames the Docsy render-heading hook and associated heading self-link helper to enable a cleaner refactoring and update the relevant documentation.
- Renames the partial from the old template naming to a new partial-based naming scheme.
- Updates documentation and CHANGELOG to reflect these renames.
Reviewed Changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
File | Description |
---|---|
userguide/layouts/_default/_markup/render-heading.html | Updated to call the new partial "td/render-heading.html". |
userguide/content/en/docs/adding-content/navigation.md | Documentation changes to reflect the new partial names. |
layouts/partials/td/render-heading.html | Renamed partial invocation & definition for heading self-link. |
CHANGELOG.md | Updates reflecting the breaking changes in naming. |
Files not reviewed (1)
- userguide/static/refcache.json: Language not supported
4f84770
to
05e8446
Compare
05e8446
to
66e8a56
Compare
Merging. Feedback is still welcome. |
Successfully tested via OTel website: |
Looks much clearer, thanks! |
There's a `no such template "_default/_markup/td-render-heading.html"` error when running `hugo server` locally. This is due to a breaking change in [Hugo v0.146.0](https://github.com/gohugoio/hugo/releases/tag/v0.146.0). This PR updates the render-heading file according to google/docsy#2223.
There's a `no such template "_default/_markup/td-render-heading.html"` error when running `hugo server` locally. This is due to a breaking change in [Hugo v0.146.0](https://github.com/gohugoio/hugo/releases/tag/v0.146.0). This PR updates the render-heading file according to google/docsy#2223. 9ecf175
There's a `no such template "_default/_markup/td-render-heading.html"` error when running `hugo server` locally. This is due to a breaking change in [Hugo v0.146.0](https://github.com/gohugoio/hugo/releases/tag/v0.146.0). This PR updates the render-heading file according to google/docsy#2223. 9ecf175
Uh oh!
There was an error while loading. Please reload this page.