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 SEO issues reported by ahrefs #38423

Merged
merged 1 commit into from
Aug 15, 2023

Conversation

@oliviertassinari oliviertassinari added the docs Improvements or additions to the documentation label Aug 11, 2023
Comment on lines -11 to -15
### Discord

MUI maintains an official [Discord server](https://mui.com/r/discord/) for real-time updates and discussions.
This is a great place to post if you need help troubleshooting or you're looking for feedback on a project.

Copy link
Member Author

Choose a reason for hiding this comment

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

Comment on lines -19 to +14
Head here to see if your problem has already been resolved by the community, or else [post a question](https://stackoverflow.com/questions/tagged/mui) to get help from community experts as well as MUI maintainers.
Head here to see if your problem has already been resolved by the community, or else [post a question](https://stackoverflow.com/questions/tagged/material-ui) to get help from community experts as well as MUI maintainers.
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -101,7 +101,7 @@ const LinkBehavior = React.forwardRef((props, ref) => (

The [example folder](https://github.com/mui/material-ui/tree/HEAD/examples/material-ui-nextjs-ts) provides an adapter for the use of [Next.js's Link component](https://nextjs.org/docs/api-reference/next/link) with Material UI.

- The first version of the adapter is the [`NextLinkComposed`](https://github.com/mui/material-ui/blob/-/examples/material-ui-nextjs-ts/src/Link.tsx) component.
- The first version of the adapter is the [`NextLinkComposed`](https://github.com/mui/material-ui/blob/-/examples/material-ui-nextjs-pages-router-ts/src/Link.tsx) component.
Copy link
Member Author

Choose a reason for hiding this comment

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

Broken link

@oliviertassinari oliviertassinari merged commit aa842c7 into mui:master Aug 15, 2023
28 checks passed
@oliviertassinari oliviertassinari deleted the fix-seo-issues-v9 branch August 15, 2023 11:45
@oliviertassinari
Copy link
Member Author

oliviertassinari commented Aug 15, 2023

I removed the Discord link from the support section. This feels much better without it.

@richbustos in terms of Discord adoption, people searching for MUI discord will find it from the first results in Google, we also have the footer icon, so I think we are pretty much good with it. Why not add a community page as well. Then, I think it's enough, we would be matching Next.js approach. I don't think that the size of the Discord channel matters so much, having engaged users feels a lot more important, so a little bit of friction to join can serve as a great filter.

Overall, I think we can move our focus to something else, I don't think we should spend more than 1h/day on Discord, we can edge our bets, we could invest on social proofs (case study, online reviews, showcase), content (new comparison pages, SEO blog post, a short presentation video), etc.

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.

None yet

2 participants