-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
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
[docs] Add redirect for typography migration #16077
Conversation
# link shortener | ||
/r/styles-instance-warning https://material-ui.com/getting-started/faq#i-have-several-instances-of-styles-on-the-page 307 | ||
/r/caveat-with-refs-guide https://material-ui.com/guides/composition/#caveat-with-refs 307 | ||
/r/pseudo-classes-guide https://material-ui.com/guides/composition/#caveat-with-refs 307 |
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.
https://en.wikipedia.org/wiki/List_of_HTTP_status_codes#3xx_Redirection
Since it might change 301 seems like the wrong choice. It should be switched to 301 if it actually points to a specific version of the docs i.e. the url is no longer used in new code versions.
Details of bundle changes.Comparing: b185d46...0e44f36
|
Links in code should be perma links which we control. Once the link becomes outdated we can just switch the redirect which ensures that the links are still valid in older versions of the code.
Closes #16011
/cc @Pajn