Skip to content

Conversation

0xlukem
Copy link
Contributor

@0xlukem 0xlukem commented Sep 18, 2025

Description

This PR updates some old links according to the Semrush report and the integrity check.

Checklist

  • Added a label 🏷️ to this PR
  • Ran my changes through Grammarly
  • Added a disclaimer if required
  • If pages were moved, opened a corresponding PR in moonbeam-mkdocs to update redirects

Translations

Does this PR update a page that also exists on the Chinese docs site? See mapping to confirm.

  • Yes
  • No

If Yes, complete the following:

  • Opened a PR on the Chinese docs repo for the corresponding page
  • Updated images, snippets, or variables if they were moved, renamed, or deleted

Link to the corresponding CN docs PR: <INSERT_LINK>

@0xlukem 0xlukem requested a review from a team as a code owner September 18, 2025 15:30
@0xlukem 0xlukem 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 labels Sep 18, 2025
@Copilot Copilot AI review requested due to automatic review settings September 18, 2025 15:30
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates various external links across the documentation according to SEO analysis from Semrush and integrity checks. The changes focus on correcting outdated URLs and ensuring links point to current, accessible pages.

  • Updated GitHub repository paths for ERC-20 interfaces from deprecated locations
  • Fixed OnFinality website URLs to include proper language paths (/en)
  • Corrected dRPC.org chainlist URLs for Moonbeam RPC endpoints
  • Updated Kusama documentation links to current URL structure
  • Fixed Moonbeam forum links by removing "status-idea" from proposal URLs
  • Corrected Trezor documentation links to current guide structure

Reviewed Changes

Copilot reviewed 27 out of 27 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tokens/connect/trezor.md Updated Trezor passphrase documentation link
llms-full.txt Comprehensive link updates across all documentation content
llms-files/*.txt Applied same link corrections to all segmented documentation files
learn/features/randomness.md Fixed Kusama epoch documentation link
learn/core-concepts/tx-fees.md Updated Moonbeam forum proposal link
builders/interoperability/xcm/xc20/interact.md Corrected GitHub ERC-20 interface path
builders/interoperability/xcm/overview.md Updated ERC-20 interface GitHub link
builders/get-started/endpoints.md Fixed OnFinality and dRPC service links

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

Style Guide Violations Summary

builders/get-started/endpoints.md (Line 132)

  • Issue:
    Missing hyphens in compound adjectives

  • Correction:

[OnFinality](https://onfinality.io/en){target=_blank} provides a free API key-based endpoint for customers in place of a public endpoint. Additionally, OnFinality offers paid tiers of service that offer increased rate limits and higher performance than those offered by the free tier. You also receive more in-depth analytics of the usage of your application.

builders/interoperability/xcm/overview.md (Line 67)

  • Issue:
    The GitHub URL path should use 'assets-erc20' instead of 'balances-erc20' based on Moonbeam's repository structure

  • Correction:

To this end, Moonbeam has introduced [XC-20s](/builders/interoperability/xcm/xc20/overview/){target=_blank}, which expand on Moonbeam's unique Ethereum compatibility features. XC-20s allow Polkadot native assets to be represented via a standard [ERC-20 interface](https://github.com/moonbeam-foundation/moonbeam/blob/master/precompiles/assets-erc20/ERC20.sol){target=_blank} through a precompiled contract. When these assets are registered on Moonbeam, they can be set as XCM execution fee assets. Consequently, when a user transfers such an asset to Moonbeam, a small part of the amount will be used to cover the XCM execution fees.

builders/interoperability/xcm/xc20/interact.md (Line 21)

  • Issue:
    File names in links should use inline code formatting per style guide

  • Correction:

As mentioned, you can interact with XC-20s via an ERC-20 interface. The [`ERC20.sol`](https://github.com/moonbeam-foundation/moonbeam/blob/master/precompiles/balances-erc20/ERC20.sol){target=_blank} interface on Moonbeam follows the [EIP-20 Token Standard](https://eips.ethereum.org/EIPS/eip-20){target=_blank}, which is the standard API interface for tokens within smart contracts. The standard defines the required functions and events that a token contract must implement to be interoperable with different applications.

builders/interoperability/xcm/xc20/interact.md (Line 125)

  • Issue:
    File names in links should use inline code formatting per style guide

  • Correction:

1. Get a copy of [`ERC20.sol`](https://github.com/moonbeam-foundation/moonbeam/blob/master/precompiles/balances-erc20/ERC20.sol){target=_blank}

learn/core-concepts/tx-fees.md (Line 19)

  • Issue:
    Missing space before link target attribute

  • Correction:

- The [dynamic fee mechanism](https://forum.moonbeam.network/t/proposal-dynamic-fee-mechanism-for-moonbeam-and-moonriver/241) {target=_blank} resembles that of [EIP-1559](https://eips.ethereum.org/EIPS/eip-1559) {target=_blank} but the implementation is different

learn/core-concepts/tx-fees.md (Line 292)

  • Issue:
    Missing space before link target attribute

  • Correction:

The `BaseFee` is the minimum amount charged to send a transaction and is a value set by the network itself. It was introduced in [EIP-1559](https://eips.ethereum.org/EIPS/eip-1559) {target=_blank}. Moonbeam has its own [dynamic fee mechanism](https://forum.moonbeam.network/t/proposal-dynamic-fee-mechanism-for-moonbeam-and-moonriver/241) {target=_blank} for calculating the base fee, which is adjusted based on block congestion. As of runtime 2300, the dynamic fee mechanism has been rolled out to all of the Moonbeam-based networks.

learn/features/randomness.md (Line 75)

  • Issue:
    Numbers below 10 should be spelled out per style guide

  • Correction:

BABE epoch randomness is based on a hash of the VRF values from the blocks produced in the relay chain epoch before last. On Polkadot, an [epoch lasts for roughly four hours](https://wiki.polkadot.com/learn/learn-cryptography/#vrf){target=_blank}, and on Kusama, an [epoch lasts for roughly one hour](https://wiki.polkadot.com/kusama/kusama-getting-started/#periods-of-common-actions-and-attributes){target=_blank}. The hashing is completed on the relay chain, and as such, it is not possible for a collator on Moonbeam to influence the randomness value unless they are also a validator on the relay chain and were responsible for producing the last output included in an epoch.

tokens/connect/trezor.md (Line 65)

  • Issue:
    Wordy phrasing in parenthetical description; violates 'Keep sentences short and to the point' guideline

  • Correction:

Next, you are asked if you want to export the public key of your Ethereum account (tab 1 in image). Right after, you are prompted with an option to use Trezor's passphrase (tab 2 in image). If you want to use the default wallet, just press **Enter**. If not, please follow [Trezor's guide article for passphrase wallets](https://trezor.io/guides/backups-recovery/advanced-wallets/what-is-a-passphrase){target=_blank}.

themacexpert
themacexpert previously approved these changes Sep 18, 2025
Copy link
Contributor

@themacexpert themacexpert left a comment

Choose a reason for hiding this comment

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

All the links look good to me!

Copy link
Contributor

@themacexpert themacexpert 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 added B1 - Ready to Merge Pull request is ready to be merged and removed B0 - Needs Review Pull request is ready for review labels Sep 30, 2025
@eshaben eshaben merged commit f93f8ea into master Sep 30, 2025
3 checks passed
@eshaben eshaben deleted the 0xlukem/sep-seo branch September 30, 2025 21:12
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 Merge Pull request is ready to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants