Skip to content

[docs-infra] Fix blog SEO for pagination #37926

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

Closed
wants to merge 3 commits into from

Conversation

alexfauquette
Copy link
Member

@alexfauquette alexfauquette commented Jul 11, 2023

@alexfauquette alexfauquette added scope: docs-infra Specific to the docs-infra product website Pages that are not documentation-related, marketing-focused. blog labels Jul 11, 2023
@mui-bot
Copy link

mui-bot commented Jul 11, 2023

Netlify deploy preview

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

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against d0927c7

Copy link
Member

@oliviertassinari oliviertassinari left a comment

Choose a reason for hiding this comment

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

To fully help the bots and the UX, each blog page should be server-side rendered. https://deploy-preview-37926--material-ui.netlify.app/blog/?page=5 server response is still the one of the first page. Unless we do it like Vercel and Hashicorp, they have a load more button for their blog, not a pagination, and reference all the blog posts in their sitemap, e.g. https://vercel.com/crawled-sitemap.xml.

const hrefWithSpecificPage = (newPage: number) => {
const tags = getTags();

return `/blog/?${[
Copy link
Member

@oliviertassinari oliviertassinari Jul 11, 2023

Choose a reason for hiding this comment

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

Clicking on the first page should ideally not point to https://deploy-preview-37926--material-ui.netlify.app/blog/?. It should point to https://deploy-preview-37926--material-ui.netlify.app/blog/. This is to have more unicity of the URLs, hence less crawling budget spent on pages.

Screenshot 2023-07-11 at 18 34 30

@oliviertassinari oliviertassinari added the bug 🐛 Something doesn't work label Jul 11, 2023
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Jul 18, 2023
@alexfauquette
Copy link
Member Author

I'm wondering if have a site map would not be another solution allowing google to simply crawl pages and even saving some crawl budget since blog posts are rarely updated

@oliviertassinari
Copy link
Member

oliviertassinari commented Feb 22, 2024

I'm wondering if have a site map would not be another solution allowing google to simply crawl pages and even saving some crawl budget since blog posts are rarely updated

@alexfauquette A sitemap could solve the SEO problem. Maybe we could do it the same way as for RSS: #35777.

Some blog benchmark: https://www.notion.so/mui-org/docs-infra-Blog-52fdceef9eb04402a4ae99c845b98486.

@alexfauquette
Copy link
Member Author

Thanks for pointing to the RSS

I was also wondering if a website could have multiple sitemap, it seems that yes

@danilo-leal danilo-leal removed the website Pages that are not documentation-related, marketing-focused. label Mar 5, 2024
@siriwatknp
Copy link
Member

What's the status of this PR?

@alexfauquette
Copy link
Member Author

The will need another PR. I'm closing this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blog bug 🐛 Something doesn't work PR: out-of-date The pull request has merge conflicts and can't be merged scope: docs-infra Specific to the docs-infra product
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[blog] Blog pages are not indexable
5 participants