Skip to content

Add glossary hook#1686

Merged
eshaben merged 4 commits into
masterfrom
0xlukem/glossary-hook
May 27, 2026
Merged

Add glossary hook#1686
eshaben merged 4 commits into
masterfrom
0xlukem/glossary-hook

Conversation

@0xlukem
Copy link
Copy Markdown
Contributor

@0xlukem 0xlukem commented May 22, 2026

📝 Description

This PR removes urls from the glossary to start using the new glossary hook
This PR needs to be merged with this one: papermoonio/polkadot-mkdocs#270

🔍 Review Preference

Choose one:

  • ✅ I have time to handle formatting/style feedback myself
  • ⚡ Docs team handles formatting (check "Allow edits from maintainers")

✅ Checklist

@0xlukem 0xlukem requested a review from a team as a code owner May 22, 2026 22:13
Copilot AI review requested due to automatic review settings May 22, 2026 22:13
@0xlukem 0xlukem added B0 - Needs Review Pull request is ready for review A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) labels May 22, 2026
@0xlukem 0xlukem requested a review from eshaben May 22, 2026 22:15
Copy link
Copy Markdown
Contributor

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 documentation pages to stop using direct /reference/glossary#... links in favor of an upcoming “glossary hook” mechanism (noted as needing to merge with papermoonio/polkadot-mkdocs#270).

Changes:

  • Removed inline glossary URL links across multiple docs pages and replaced them with plain term mentions.
  • Adjusted term casing in several places (e.g., “Chain Specification”, “Existential Deposit”, “Full Node”, “Pallet”, “Weight”) to align with glossary term names.
  • Minor formatting/cleanup changes (e.g., closing </div> alignment, trailing blank line removal).

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
smart-contracts/for-eth-devs/evm-vs-pvm.md Removes direct glossary link for PVM references.
reference/tools/omninode.md Replaces glossary link for “chain specification” with plain term text.
reference/tools/light-clients.md Removes glossary links and updates casing for several glossary terms.
reference/polkadot-hub/consensus-and-security/relay-chain.md Removes glossary link for NPoS mention in intro text.
reference/polkadot-hub/consensus-and-security/pos-consensus.md Removes glossary link for NPoS mention; trims trailing blank line.
reference/polkadot-hub/bridging.md Removes glossary link for “pallets” mention and replaces with plain term text.
reference/polkadot-hub/assets.md Removes glossary link for “existential deposit” mention; minor closing div formatting.
reference/parachains/index.md Removes glossary link for “pallets” mention and replaces with plain term text.
reference/parachains/consensus/index.md Removes glossary link for NPoS mention and replaces with plain term text.
reference/parachains/blocks-transactions-fees/fees.md Removes glossary link for “Weights” mention; list formatting tweak at end.
reference/index.md Removes glossary links for relay chain/parachain/governance mentions and replaces with plain text.
parachains/runtime-maintenance/coretime-renewal.md Removes glossary link for “sovereign account” mention and replaces with plain text.
parachains/launch-a-parachain/deploy-to-polkadot.md Removes glossary links/mentions for collators/chain spec; updates casing in a few places.
parachains/interoperability/get-started.md Removes glossary link for “weight” mention and replaces with plain text.
parachains/customize-runtime/pallet-development/pallet-testing.md Removes glossary link for FRAME mention and replaces with plain text.
parachains/customize-runtime/index.md Removes glossary link for FRAME mention; minor list formatting tweak.
chain-interactions/send-transactions/interoperability/transfer-assets-parachains.md Removes glossary links for existential deposit mentions and replaces with plain text.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread reference/polkadot-hub/consensus-and-security/relay-chain.md
Comment thread reference/parachains/index.md
Comment thread reference/tools/light-clients.md
Comment thread reference/tools/light-clients.md
Comment thread reference/polkadot-hub/bridging.md
Comment thread parachains/interoperability/get-started.md
Comment thread parachains/interoperability/get-started.md
Comment thread parachains/launch-a-parachain/deploy-to-polkadot.md Outdated
Comment thread parachains/launch-a-parachain/deploy-to-polkadot.md
@0xlukem 0xlukem requested a review from eshaben May 26, 2026 19:11
@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 May 27, 2026
@eshaben eshaben merged commit e156848 into master May 27, 2026
11 of 13 checks passed
@eshaben eshaben deleted the 0xlukem/glossary-hook branch May 27, 2026 20:25
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.

4 participants