Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Fixing Helm operator docs link #2863

Merged
merged 1 commit into from
Feb 21, 2020

Conversation

mattfarina
Copy link
Contributor

The link of https://docs.fluxcd.io/projects/helm-operator/ is a
redirect to https://docs.fluxcd.io/projects/helm-operator/[LANG]/[VERSION]/
for example you get a link like
https://docs.fluxcd.io/projects/helm-operator/en/1.0.0-rc9/

But, when the trailing / is missing the redirect does not happen
and the user gets a 404.

The link of https://docs.fluxcd.io/projects/helm-operator/ is a
redirect to https://docs.fluxcd.io/projects/helm-operator/[LANG]/[VERSION]/
for example you get a link like
https://docs.fluxcd.io/projects/helm-operator/en/1.0.0-rc9/

But, when the trailing / is missing the redirect does not happen
and the user gets a 404.

Signed-off-by: Matt Farina <[email protected]>
Copy link
Member

@hiddeco hiddeco left a comment

Choose a reason for hiding this comment

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

I think RTD has changed their redirect rules recently, as I am quite sure this worked when I put it there.

Nevertheless thanks for catching this 🌞

@hiddeco hiddeco added the docs Issue or PR relates to documentation label Feb 20, 2020
@stefanprodan stefanprodan merged commit 09a42bf into fluxcd:master Feb 21, 2020
@hiddeco hiddeco added this to the 1.19.0 milestone Feb 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
docs Issue or PR relates to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants