(#2664) Add redirects for pages that have moved #2794
Merged
Conversation
This file contains 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
As a result of changes to the structure of the documentation site, the original 301 redirects from readthedocs, were resulting in 404's within GitHub pages. This PR addresses this by ensuring that a Wyam RedirectFrom attribute is added to all affected pages, so that the end result is arrive on the correct page. There may be additional changes that need to be made to capture all pages, but this commit contains all 301 redirects from the current readthedocs configuration.
arturcic
approved these changes
Aug 2, 2021
|
Thank you @gep13 for your contribution! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
As a result of changes to the structure of the documentation site, the
original 301 redirects from readthedocs, were resulting in 404's within
GitHub pages. This PR addresses this by ensuring that a Wyam
RedirectFrom attribute is added to all affected pages, so that the end
result is arrive on the correct page.
There may be additional changes that need to be made to capture all
pages, but this commit contains all 301 redirects from the current
readthedocs configuration.
Related Issue
#2664
Motivation and Context
Attempts to address broken page links from Google Searches, as well as any existing saved URL's to old readthedocs pages.
How Has This Been Tested?
Went through each page during live stream on Twitch, using the preview-documentation Cake Task. Witnessed by @arturcic who was present on the stream 😄
Screenshots (if appropriate):
Checklist: