-
-
Notifications
You must be signed in to change notification settings - Fork 57
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
chore: added redirects #168
Conversation
src/static/_redirects
Outdated
/docs https://eslint.org/ 301! | ||
/docs/user-guide/rules https://eslint.org/docs/rules 301! | ||
/docs/rules/id-blacklist https://eslint.org/docs/rules/id-denylist 301! | ||
|
||
/docs/0.24.1/command-line-interface https://eslint.org/docs/user-guide/command-line-interface 301! | ||
/docs/0.24.1/configuring https://eslint.org/docs/user-guide/configuring 301! |
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.
I believe we don't need these internal redirects for docs since we will have a separate website for docs.
/cc @nzakas can you confirm?
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.
I believe we don't need these internal redirects for docs since we will have a separate website for docs.
Thought we might later on update the links. For now wanted to add proper working links
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.
For now all we need is /docs/* redirecting to eslint.org/docs/*
Later we can change to docs.eslint.org/*
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.
done
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.
Just need to be sure the docs go to the correct location.
src/static/_redirects
Outdated
/docs/* https://eslint.org/ 301! |
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.
/docs/* https://eslint.org/ 301! | |
/docs/* https://eslint.org/docs/:splat 302! |
We need to cover all docs URLs and definitely don’t want this to be permanent.
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.
Done
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.
LGTM, thanks for contributing.
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.
LGTM. Thanks!
Addresses: