Skip to content

[docs] Add a new section for Integration #4411

Merged
prakhargupta1 merged 11 commits intomui:masterfrom
prakhargupta1:integration-docs
Nov 22, 2024
Merged

[docs] Add a new section for Integration #4411
prakhargupta1 merged 11 commits intomui:masterfrom
prakhargupta1:integration-docs

Conversation

@prakhargupta1
Copy link
Contributor

@prakhargupta1 prakhargupta1 commented Nov 13, 2024

@prakhargupta1 prakhargupta1 added the docs Improvements or additions to the documentation. label Nov 13, 2024
@prakhargupta1 prakhargupta1 linked an issue Nov 13, 2024 that may be closed by this pull request
@prakhargupta1
Copy link
Contributor Author

@bharatkashyap Not sure why Netlify preview fails. Can you have a look?

@bharatkashyap
Copy link
Collaborator

Instead of a root level section, can we make it a sub-section of the Introduction? I think that makes more sense in the hierarchy of the navigation, what do you think?

Copy link
Collaborator

@apedroferreira apedroferreira left a comment

Choose a reason for hiding this comment

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

Some Vale changes in my own copy :D

Also why did we remove the numbers from the headings?
In the React Router guide I sometimes mention "step 1" or "step 3" but this way it doesn't make sense anymore, we'd have to change it.

@prakhargupta1
Copy link
Contributor Author

Also why did we remove the numbers from the headings?
In the React Router guide I sometimes mention "step 1" or "step 3" but this way it doesn't make sense anymore, we'd have to change it.

Based on the core docs Integration section, I removed the numbers. There were two mentions of step 1, step 2 which I have now tackled.

Copy link
Collaborator

@apedroferreira apedroferreira left a comment

Choose a reason for hiding this comment

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

Thanks Prakhar!

@prakhargupta1 prakhargupta1 merged commit e189558 into mui:master Nov 22, 2024
@prakhargupta1 prakhargupta1 deleted the integration-docs branch November 22, 2024 15:50
@bharatkashyap
Copy link
Collaborator

bharatkashyap commented Nov 26, 2024

Instead of a root level section, can we make it a sub-section of the Introduction? I think that makes more sense in the hierarchy of the navigation, what do you think?

We should have also added new redirect rules to the _redirects file given that we went ahead with a new root level section. The old URLs are now returning 404s: https://mui.com/toolpad/core/introduction/integration/

@prakhargupta1 Can you take this up? I'll redeploy the docs once this is merged

@prakhargupta1
Copy link
Contributor Author

prakhargupta1 commented Nov 26, 2024

Oh sorry, completely missed it. Thanks for pointing it out! PR: #4475

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[docs] Change the integration docs to an Integration section

3 participants