Conversation
📝 WalkthroughWalkthroughThis PR removes references to the SubQuery Network and Miscellaneous documentation sections from the VuePress site configuration, including navbar entries, sidebar navigation structure, and homepage guide listings. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docs/.vuepress/config.ts (1)
47-50:⚠️ Potential issue | 🟠 MajorSidebar wiring still exposes removed sections
getSidebar("")still serves entries for Miscellaneous and Publish to the SubQuery Network (seedocs/.vuepress/sidebar.tsLine 592-602 and Line 543-546 from provided context). That means users can still navigate to the sections this PR aims to remove.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/.vuepress/config.ts` around lines 47 - 50, The sidebar still includes the removed "Miscellaneous" and "Publish to the SubQuery Network" sections — fix by updating the getSidebar call and implementation: add an optional excludes parameter to getSidebar in sidebar.ts (e.g., excludes: string[]), update getSidebar to filter out any sidebar entries whose title matches items in excludes, and change the config.ts call from getSidebar("") to getSidebar("", { excludes: ["Miscellaneous", "Publish to the SubQuery Network"] }) so those sections are no longer served.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In `@docs/.vuepress/config.ts`:
- Around line 47-50: The sidebar still includes the removed "Miscellaneous" and
"Publish to the SubQuery Network" sections — fix by updating the getSidebar call
and implementation: add an optional excludes parameter to getSidebar in
sidebar.ts (e.g., excludes: string[]), update getSidebar to filter out any
sidebar entries whose title matches items in excludes, and change the config.ts
call from getSidebar("") to getSidebar("", { excludes: ["Miscellaneous",
"Publish to the SubQuery Network"] }) so those sections are no longer served.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 87e08ec6-974e-44c3-bc09-45ab7cc25de2
📒 Files selected for processing (3)
docs/.vuepress/config.tsdocs/.vuepress/sidebar.tsdocs/README.md
💤 Files with no reviewable changes (2)
- docs/README.md
- docs/.vuepress/sidebar.ts
Summary by CodeRabbit