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 301 link #6239

Merged
merged 1 commit into from
Sep 22, 2022
Merged

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Sep 21, 2022

I found this in the Algolia crawl results (I was looking a bit at how we can improve the Algolia UX)

Screenshot 2022-09-21 at 18 15 13

https://crawler.algolia.com/admin/crawlers/739c29c8-99ea-4945-bd27-17a1df391902/inspector?search=&page=1&status=SKIPPED

@oliviertassinari oliviertassinari added the docs Improvements or additions to the documentation label Sep 21, 2022
@oliviertassinari oliviertassinari added the bug 🐛 Something doesn't work label Sep 21, 2022
@mui-bot
Copy link

mui-bot commented Sep 21, 2022

These are the results for the performance tests:

Test case Unit Min Max Median Mean σ
Filter 100k rows ms 491.4 862.6 684.9 654.96 142.923
Sort 100k rows ms 510.8 1,000.1 666.7 771.3 167.929
Select 100k rows ms 193.4 253.9 218.9 219.96 21.58
Deselect 100k rows ms 120 397.2 169.6 200.36 100.679

Generated by 🚫 dangerJS against addc45d

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Sep 22, 2022
@github-actions
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Sep 22, 2022
@oliviertassinari oliviertassinari merged commit 969313d into mui:next Sep 22, 2022
@oliviertassinari oliviertassinari deleted the fix-301-link branch September 22, 2022 09:44
@oliviertassinari
Copy link
Member Author

oliviertassinari commented Sep 22, 2022

Regarding why this wasn't detected by ahrefs sooner, it's because of Netlify: https://answers.netlify.com/t/no-trailing-slash-when-using-proxies/48790/18.

oliviertassinari added a commit that referenced this pull request Sep 22, 2022
See #6239 for why. It's the same fix but
on the master branch.
@cherniavskii cherniavskii mentioned this pull request Sep 22, 2022
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