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] Fix links to GitHub #538

Merged

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Nov 4, 2020

The introduction of a second GitHub repository means that we can no longer hard-code the URL in the documentation, if we do, we won't point to the right position. mui/material-ui#23390 allows configuring the underlying repository, and this PR leverage it. For instance, these two links are now correct:

Capture d’écran 2020-11-04 à 19 21 41

Capture d’écran 2020-11-04 à 19 21 47

Proof: https://deploy-preview-538--material-ui-x.netlify.app/components/data-grid/rows/

@oliviertassinari oliviertassinari added bug 🐛 Something doesn't work docs Improvements or additions to the documentation labels Nov 4, 2020
@oliviertassinari oliviertassinari merged commit b5bd56f into mui:master Nov 6, 2020
@oliviertassinari oliviertassinari deleted the docs-fix-github-links branch November 6, 2020 16:41
oliviertassinari added a commit to oliviertassinari/mui-x that referenced this pull request Nov 6, 2020
dtassone pushed a commit to dtassone/material-ui-x that referenced this pull request Nov 9, 2020
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants