Skip to content
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

Merged
merged 3 commits into from
Jun 6, 2019

Conversation

eps1lon
Copy link
Member

@eps1lon eps1lon commented Jun 5, 2019

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

@eps1lon eps1lon added docs Improvements or additions to the documentation package: material-ui Specific to @mui/material labels Jun 5, 2019
@eps1lon eps1lon requested a review from oliviertassinari June 5, 2019 19:05
# 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
Copy link
Member Author

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.

@mui-pr-bot
Copy link

Details of bundle changes.

Comparing: b185d46...0e44f36

bundle parsed diff gzip diff prev parsed current parsed prev gzip current gzip
@material-ui/core 0.00% 0.00% 318,958 318,958 87,399 87,399
@material-ui/core/Paper 0.00% 0.00% 67,892 67,892 20,188 20,188
@material-ui/core/Paper.esm 0.00% 0.00% 61,186 61,186 18,987 18,987
@material-ui/core/Popper 0.00% 0.00% 28,728 28,728 10,344 10,344
@material-ui/core/Textarea 0.00% 0.00% 5,513 5,513 2,373 2,373
@material-ui/core/TrapFocus 0.00% 0.00% 3,755 3,755 1,572 1,572
@material-ui/core/styles/createMuiTheme 0.00% 0.00% 15,944 15,944 5,793 5,793
@material-ui/core/useMediaQuery 0.00% 0.00% 2,529 2,529 1,087 1,087
@material-ui/lab 0.00% 0.00% 138,290 138,290 42,505 42,505
@material-ui/styles 0.00% 0.00% 51,384 51,384 15,186 15,186
@material-ui/system 0.00% 0.00% 14,463 14,463 4,179 4,179
Button 0.00% 0.00% 83,821 83,821 25,471 25,471
Modal 0.00% 0.00% 20,343 20,343 6,687 6,687
colorManipulator 0.00% 0.00% 3,904 3,904 1,543 1,543
docs.landing 0.00% 0.00% 55,085 55,085 13,845 13,845
docs.main -0.00% -0.01% 653,429 653,405 206,607 206,596
packages/material-ui/build/umd/material-ui.production.min.js 0.00% 0.00% 292,558 292,558 83,364 83,364

Generated by 🚫 dangerJS against 0e44f36

@mbrookes

This comment has been minimized.

@eps1lon

This comment has been minimized.

@eps1lon eps1lon merged commit 9d1d2e8 into mui:master Jun 6, 2019
@eps1lon eps1lon deleted the docs/redirect-typo-migration branch June 6, 2019 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation package: material-ui Specific to @mui/material
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Old links to Typeography v2 migration are broken
4 participants