-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 "Licensing" page link #12156
Conversation
@@ -296,7 +296,7 @@ This error indicates that your MUI X license key format isn't valid. | |||
It could be because the license key is missing a character or has a typo. | |||
|
|||
To solve the issue, you need to double-check that `setLicenseKey()` is called with the right argument. | |||
Please check the [license key installation](#license-key-installation). | |||
Please check the [license key installation](/x/introduction/licensing/#license-key). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update to full relative URL to benefit from docs:link-check
.
Deploy preview: https://deploy-preview-12156--material-ui-x.netlify.app/ Updated pages: |
This is a regression from https://github.com/mui/mui-private/pull/339 cc @hasdfa. I have opened https://github.com/mui/mui-private/issues/404. There are broken links on MUI Core, MUI X (there are more in this repo) and MUI Private repos. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
Unless you think @oliviertassinari need to approve explicitly, it would be nice to have it in today's release
Given the https://github.com/mui/mui-private/pull/405 it seems that the current solution will not be correct. 🤔 |
Signed-off-by: Lukas <llukas.tyla@gmail.com>
Removed the |
Perfect, it looks all correct 👌 |
Address docs feedback: