Skip to content

Conversation

@robobario
Copy link
Member

This change ensures that /docs/v0.13.0/* urls HTML redirect to their equivalent location under /documentation/0.13.0/

After the release we have since moved to a new mechanism where HTML docs are generated in the core repo and copied into this one. So we have two copies of the docs committed. This removes the asciidoc version.

To achieve this I changed the redirect layout so that the page.target property is expected to be a path relative to the base url (and updated the error data file for 0.13.0). Unfortunately you can't use Jekyll liquid templating in the frontmatter, so target: {{ /path/to/page | absolute_url }} is impossible.

Why:
I want to re-use the redirect layout from some HTML frontmatter, and
you cannot use liquid templating nicely in the Jekyll frontmatter to
create an absolute url like "target: {{"/path/to/target" | absolute_url}}".
I want to use root relative paths so that it works when there is a base
url.

In future the layout would need adjustment if we wanted it to handle
absolute redirect targets, maybe passing a different property to the
page, or sniffing the first N characters for http:/https:

Signed-off-by: Robert Young <[email protected]>
This change ensures that /docs/v0.13.0/* urls HTML redirect to their
equivalent location under /documentation/0.13.0/

After the release we have since moved to a new mechanism where HTML docs
are generated in the core repo and copied into this one. So we have two
copies of the docs.

Signed-off-by: Robert Young <[email protected]>
@robobario robobario requested a review from a team as a code owner July 31, 2025 02:06
Copy link
Member

@SamBarker SamBarker left a comment

Choose a reason for hiding this comment

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

LGTM

@SamBarker SamBarker merged commit 730d49d into kroxylicious:main Jul 31, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants