-
Notifications
You must be signed in to change notification settings - Fork 76
Hide pages for MVP #1251
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
Hide pages for MVP #1251
Conversation
There was a problem hiding this 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 hides documentation pages not included in the MVP release by commenting out navigation entries and moving files to hidden directories. The changes focus on removing the "Chain Interactions" and "Technical Reference" sections from the main navigation while making minor improvements to page titles and headings for consistency.
Key Changes:
- Commented out "Chain Interactions" and "Technical Reference" sections in the main navigation
- Moved reference and chain-interactions content to hidden directories (prefixed with
.) - Updated page titles to be more descriptive (e.g., "Faucet" → "Get Tokens from the Official Faucet")
- Standardized heading usage (e.g., "## Overview" → "## Introduction")
- Updated skip_paths configuration to exclude hidden directories from LLM processing
Reviewed Changes
Copilot reviewed 83 out of 163 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
.nav.yml |
Commented out Chain Interactions and Technical Reference navigation entries |
smart-contracts/integrations/.nav.yml |
Commented out Oracles and Indexers pages |
smart-contracts/faucet.md |
Updated title to be more descriptive |
smart-contracts/for-eth-devs/gas-model.md |
Changed "Overview" to "Introduction" |
smart-contracts/cookbook/.nav.yml |
Renamed "EVM Smart Contracts" to "Build Smart Contracts" |
smart-contracts/cookbook/dapps/zero-to-hero.md |
Fixed "dApp" capitalization to "DApp" |
parachains/.nav.yml |
Updated navigation labels for better clarity |
parachains/testing/*.md |
Updated titles to match navigation entries |
parachains/interoperability/get-started.md |
Updated title and heading from "Introduction to XCM" to "Get Started with XCM" |
.reference/**/*.md |
Moved all reference pages to hidden directory |
.chain-interactions/**/*.md |
Moved all chain-interactions pages to hidden directory |
.ai/pages/**/*.md |
Updated AI-generated page paths and removed hidden content |
scripts/llms_config.json |
Added hidden directories to skip_paths |
llms.txt |
Updated with reduced page count and removed hidden content references |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
0xlukem
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
nhussein11
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! many thanks
* hide content for mvp launch * update llms config to skip chain interactions and reference sections * llms
* add benchmark your pallet * cleanup and grammarly * fresh llms * apply copilot feedback * update snippet title * Apply suggestions from code review Co-authored-by: Erin Shaben <eshaben@icloud.com> * apply feedback from review * Add Hardhat Get Started (new IA) (#1196) * add harthdat get started * fix: llms * fix and formating * fix --------- Co-authored-by: 0xlukem <lucas.malizia27@gmail.com> * Fix local dev node (#1226) * Remove outdated warning reference from local development node documentation * fix: llms * fix: llms --------- Co-authored-by: Erin Shaben <eshaben@icloud.com> * Add Remix IDE (new IA) (#1174) * fix: get started ready * fix: remix deploy * Update troubleshooting guide for Remix IDE and comment out 'Verify a Contract' section in navigation * fix: images and names * fix: remove verify contract file * fix: llms * Apply suggestions from code review Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * fix: prerequisite * fix: llms * Grammarly things and reformat to FAQ style dropdowns for troubleshooting page * applies review feedback * Clean up formatting of titles and bullets * fix: feedback * fix: llms * fix: gifs lists wording * fix: llms * Remove outdated troubleshooting and verification guides for Remix IDE on Polkadot Hub. * fix errant merge conflict syntax, description length * applies review feedback * grammarly pass * minor fix and llms --------- Co-authored-by: Dawn Kelly <83190195+dawnkelly09@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: DAWN KELLY <dawnkelly09@gmail.com> Co-authored-by: Erin Shaben <eshaben@icloud.com> * Hide pages for MVP (#1251) * hide content for mvp launch * update llms config to skip chain interactions and reference sections * llms * FIX - wallets polkavm disclaimer removed (#1252) * fix: removing polkavm disclaimer * fix: llms * applies review feedback * fresh llms --------- Co-authored-by: DAWN KELLY <dawnkelly09@gmail.com> Co-authored-by: Dawn Kelly <83190195+dawnkelly09@users.noreply.github.com> Co-authored-by: Erin Shaben <eshaben@icloud.com> Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com> Co-authored-by: 0xlukem <lucas.malizia27@gmail.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
📝 Description
This PR hides pages we aren't launching with for the MVP, including:
It also makes a couple minor changes I saw when going through some of the pages, like: