Skip to content

[docs-infra] Resolve component API URLs for Data Grid#44992

Merged
kenanyusuf merged 2 commits intomui:masterfrom
kenanyusuf:resolve-data-grid-api-url
Jan 13, 2025
Merged

[docs-infra] Resolve component API URLs for Data Grid#44992
kenanyusuf merged 2 commits intomui:masterfrom
kenanyusuf:resolve-data-grid-api-url

Conversation

@kenanyusuf
Copy link
Member

Allows us to specify components in markdown files. Currently, the URLs are resolved incorrectly.

@kenanyusuf kenanyusuf added the scope: docs-infra Involves the docs-infra product (https://www.notion.so/mui-org/b9f676062eb94747b6768209f7751305). label Jan 10, 2025
@mui-bot
Copy link

mui-bot commented Jan 10, 2025

Netlify deploy preview

https://deploy-preview-44992--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 0f83bb5

@kenanyusuf kenanyusuf requested a review from a team January 10, 2025 15:59
@alexfauquette
Copy link
Member

I don't understand what would be the impact of this modification.

From what I remember, the data grid has a specific behavior, because all the API pages have demo links to the "Overview" page. Then they don't need to resolve components

@kenanyusuf
Copy link
Member Author

kenanyusuf commented Jan 13, 2025

@alexfauquette I'm trying to fix the API links at the bottom of these component pages. Currently they link to a non-existent route, and I figured this is what will fix it.

@kenanyusuf kenanyusuf merged commit 093d4f3 into mui:master Jan 13, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: docs-infra Involves the docs-infra product (https://www.notion.so/mui-org/b9f676062eb94747b6768209f7751305).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants