Skip to content

Revert "[core] Replace getInitialProps with getStaticProps"#33991

Merged
mnajdova merged 2 commits intomasterfrom
revert-33684-docs/replace-getInitialProps-with-getStaticProps
Aug 22, 2022
Merged

Revert "[core] Replace getInitialProps with getStaticProps"#33991
mnajdova merged 2 commits intomasterfrom
revert-33684-docs/replace-getInitialProps-with-getStaticProps

Conversation

@mnajdova
Copy link
Member

@mnajdova mnajdova commented Aug 19, 2022

Reverts #33684.

The PR broke the SEO structure when it comes to the translations. For example, the page https://mui.com/material-ui/api/table-pagination/ says:

<link rel="alternate" href="https://mui.com/zh/material-ui/api/table-pagination/" hrefLang="zh"/>

But this page is a 404 https://mui.com/zh/material-ui/api/table-pagination/. Chinese users will directly get a 404. The Google Search console started to raise flags about it. The root problem might be covered in vercel/next.js#15315 (comment).

@mui-bot
Copy link

mui-bot commented Aug 19, 2022

No bundle size changes

Generated by 🚫 dangerJS against 5afde48

@mnajdova mnajdova added docs Improvements or additions to the documentation. type: regression A bug, but worse, it used to behave as expected. labels Aug 19, 2022
@mnajdova mnajdova requested a review from a team August 19, 2022 13:33
@oliviertassinari oliviertassinari added the type: bug It doesn't behave as expected. label Aug 19, 2022
@oliviertassinari oliviertassinari marked this pull request as ready for review August 19, 2022 23:30
@mnajdova mnajdova merged commit bfc3ef0 into master Aug 22, 2022
@mnajdova mnajdova deleted the revert-33684-docs/replace-getInitialProps-with-getStaticProps branch August 22, 2022 10:13
daniel-rabe pushed a commit to daniel-rabe/material-ui that referenced this pull request Nov 29, 2022
@oliviertassinari oliviertassinari removed the type: bug It doesn't behave as expected. label Apr 30, 2025
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. type: regression A bug, but worse, it used to behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants