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

[core] Fix scroll restoration in the docs #866

Merged
merged 1 commit into from Sep 19, 2022

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Aug 28, 2022

https://deploy-preview-866--mui-toolpad-docs.netlify.app/toolpad/getting-started/overview/

@@ -61,7 +61,7 @@
"lz-string": "^1.4.4",
"markdown-to-jsx": "^7.1.7",
"marked": "^4.0.17",
"next": "12.2.0",
"next": "^12.2.5",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have two different Next.js app (one for the docs, one for toolpad), I'm keeping this one in sync with the docs-infra not toolpad. I think that it's beneficial to have two different Next.js versions to make sure they are decoupled.

Copy link
Member

@Janpot Janpot Aug 29, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps we should decouple @mui/monorepo as well?

@oliviertassinari oliviertassinari added core Infrastructure work going on behind the scenes docs Improvements or additions to the documentation labels Aug 28, 2022
@oliviertassinari oliviertassinari marked this pull request as ready for review August 28, 2022 10:18
@oliviertassinari
Copy link
Member Author

I have found an issue with this approach, it leads to a warning on the core's side. We need to change the import a bit.

@oliviertassinari oliviertassinari added the on hold There is a blocker, we need to wait label Aug 29, 2022
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Sep 1, 2022
docs/next.config.js Outdated Show resolved Hide resolved
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Sep 17, 2022
@oliviertassinari oliviertassinari force-pushed the update-mono-repo branch 2 times, most recently from 41d898d to f7f1362 Compare September 17, 2022 15:22
@oliviertassinari oliviertassinari removed the on hold There is a blocker, we need to wait label Sep 17, 2022
@oliviertassinari oliviertassinari merged commit 887585c into mui:master Sep 19, 2022
@oliviertassinari oliviertassinari deleted the update-mono-repo branch September 19, 2022 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Infrastructure work going on behind the scenes docs Improvements or additions to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants