adding redirect rules to prevent google links from reaching page not …#747
Conversation
…found Signed-off-by: Chance Whittaker <chance19@gmail.com>
|
✔️ Deploy Preview for cert-manager-website ready! 🔨 Explore the source changes: 42d3bff 🔍 Inspect the deploy log: https://app.netlify.com/sites/cert-manager-website/deploys/61841a89d1f69d000746b82b 😎 Browse the preview: https://deploy-preview-747--cert-manager-website.netlify.app |
|
CC @mattbates for visibility |
Signed-off-by: Chance Whittaker <chance19@gmail.com>
Signed-off-by: Chance Whittaker <chance19@gmail.com>
|
Three commits:
|
|
Happy to review this once the conflicts have been resolved 👍 |
SgtCoDFish
left a comment
There was a problem hiding this comment.
This looks good, there's just one little thing I've raised.
There are a lot of references to docs.cert-manager.io if I do grep -R "docs\.cert-manager\.io" . in the website repo, but most are in old docs and they should be caught by the redirect rules we're adding here 👍
Signed-off-by: Chance Whittaker <chance19@gmail.com>
Signed-off-by: Chance Whittaker <chance19@gmail.com>
99d8e27 to
42d3bff
Compare
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: chance19, SgtCoDFish The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…found
Signed-off-by: Chance Whittaker chance19@gmail.com
Currently some links from Google, such as https://docs.cert-manager.io/en/release-0.11/ would route to https://cert-manager.io/docs/en/release-0.11/ which would take the browser to a 404 page not found (see screenshot)

This change should ensure that these links are at least routed back to the specific release-note page that was referenced in the link. This is not a full solution to the problem just an improvement on the current situation.