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

Site health improvements - january report #1076

Merged

Conversation

0xlukem
Copy link
Contributor

@0xlukem 0xlukem commented Jan 15, 2025

Description

Based on the January SEMrush report, I updated the Moonbeam-Docs site health by addressing several issues, depreciating one content page, and fixing several broken links.

Checklist

  • I have added a label to this PR 🏷️
  • I have run my changes through Grammarly
  • If pages have been moved around, I have created an additional PR in moonbeam-mkdocs to update redirects
  • If images have been added, I have run the compress-images.py script to compress the images.
  • If this page requires a disclaimer, I have added one

@eshaben
Copy link
Contributor

eshaben commented Jan 17, 2025

@0xlukem can you please when you open a PR, remove the translation parts of the checklist if it's not applicable. It just makes it easier for reviews to see ok, yes I need to worry about the CN site changes too or no. Also, it forces you to see if you need to worry about these things too haha 🙂

I don't know if you've ever even seen the chinese docs repo haha, but if not, here is a link: https://github.com/moonbeam-foundation/moonbeam-docs-cn

Copy link
Contributor

@eshaben eshaben left a comment

Choose a reason for hiding this comment

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

Nice finds! Just a few minor suggestions 🙂

builders/integrations/oracles/chainlink.md Outdated Show resolved Hide resolved
builders/integrations/oracles/chainlink.md Outdated Show resolved Hide resolved
node-operators/oracle-nodes/node-chainlink.md Outdated Show resolved Hide resolved
node-operators/oracle-nodes/node-chainlink.md Outdated Show resolved Hide resolved
@eshaben eshaben added A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) B0 - Needs Review Pull request is ready for review C1 - Medium Medium priority task labels Jan 17, 2025
0xlukem and others added 2 commits January 19, 2025 19:43
Apply changes according to feedback

Co-authored-by: Erin Shaben <[email protected]>
@eshaben eshaben added B1 - Ready to be Merged Pull request is ready to be merged and removed B0 - Needs Review Pull request is ready for review labels Jan 22, 2025
Copy link
Contributor

@eshaben eshaben left a comment

Choose a reason for hiding this comment

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

LGTM!

@eshaben eshaben merged commit e88e715 into moonbeam-foundation:master Jan 22, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) B1 - Ready to be Merged Pull request is ready to be merged C1 - Medium Medium priority task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants