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] Handle integration docs 404 #4475

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

prakhargupta1
Copy link
Member

@prakhargupta1 prakhargupta1 commented Nov 26, 2024

@prakhargupta1 prakhargupta1 added the docs Improvements or additions to the documentation label Nov 26, 2024
@prakhargupta1 prakhargupta1 merged commit b67c51f into mui:master Nov 26, 2024
14 checks passed
@prakhargupta1 prakhargupta1 deleted the fix-redirects branch November 26, 2024 09:55
@oliviertassinari oliviertassinari added bug 🐛 Something doesn't work regression A bug, but worse labels Nov 27, 2024
@oliviertassinari
Copy link
Member

oliviertassinari commented Nov 27, 2024

Adding a 301 is great, we need this.

Now, it's not enough to fix that regression:

Now that you understand the basic concepts of Toolpad Core, you're ready to start integrating it into your project. Head over to the [integration docs](/toolpad/core/introduction/integration/) to learn more.

there is no equivalent to this now. I guess next step should link to the tutorial, or maybe we can remove that whole "Next steps" section.

@bharatkashyap
Copy link
Member

Adding a 301 is great, we need this.

Now, it's not enough to fix that regression:

Now that you understand the basic concepts of Toolpad Core, you're ready to start integrating it into your project. Head over to the [integration docs](/toolpad/core/introduction/integration/) to learn more.

there is no equivalent to this now. I guess next step should link to the tutorial, or maybe we can remove that whole "Next steps" section.

We can remove it entirely 👍🏻 @prakhargupta1 Can you take this up? I'll redeploy once this is merged as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work docs Improvements or additions to the documentation regression A bug, but worse
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants