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

Fix SEO issues to impove the Ahrefs' Health Score #425

Merged
merged 10 commits into from
Jan 8, 2024
Merged

Conversation

nqst
Copy link
Contributor

@nqst nqst commented Jan 2, 2024

Made some changes that should improve our SEO Health Score provided by Ahrefs. As of Dec 2023, the score was 87%.

Changes:

  • Fixed broken links.
  • Added og:image and other Open Graph & Twitter tags.
  • Added meta descriptions for the website pages. I didn't add the descriptions to all blog posts, but added an example one to the most recent post.
  • Fixed the Noindex page in sitemap issue.
  • Optimized a heavyweight image.

Unrelated change: running yarn inject:protocol added some changes to the protocol that weren't added before.

@nqst nqst requested review from kvz, Acconut and nickrttn January 2, 2024 12:32
@nqst nqst self-assigned this Jan 2, 2024
Copy link
Contributor

@nickrttn nickrttn left a comment

Choose a reason for hiding this comment

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

That's amazing @nqst 👏

Is the example blog post description ok to go live with or should someone (@Acconut?) write descriptions for each post?

@nqst
Copy link
Contributor Author

nqst commented Jan 2, 2024

Is the example blog post description ok to go live?

Yeah, that's just an excerpt from the blog post introduction, so it should be good to go.

Should someone write descriptions for each post?

Ideally, yes, although that's a secondary priority compared to adding descriptions to the homepage and the pages that we have in the navbar.

Copy link
Member

@kvz kvz left a comment

Choose a reason for hiding this comment

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

💯💯💯💯
✨✨💖✨
💯💯💯💯

Copy link
Member

@Acconut Acconut left a comment

Choose a reason for hiding this comment

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

That's a great change, thank you very much, Alex!

scripts/inject-protocol.sh Outdated Show resolved Hide resolved
src/content/protocols/0.1.x.md Outdated Show resolved Hide resolved
src/content/protocols/0.2.x.md Outdated Show resolved Hide resolved
src/content/protocols/1.0.x.md Outdated Show resolved Hide resolved
scripts/inject-protocol.sh Outdated Show resolved Hide resolved
src/pages/demo.mdx Outdated Show resolved Hide resolved
src/pages/faq.md Outdated Show resolved Hide resolved
src/pages/implementations.astro Outdated Show resolved Hide resolved
src/pages/index.astro Outdated Show resolved Hide resolved
src/pages/support.mdx Outdated Show resolved Hide resolved
@Acconut Acconut merged commit b23fff1 into main Jan 8, 2024
1 check passed
@Acconut Acconut deleted the fix-seo-issues branch January 8, 2024 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants