Skip to content

feat: remove network and miscellaneous#675

Open
HuberTRoy wants to merge 1 commit intomasterfrom
feat/remove
Open

feat: remove network and miscellaneous#675
HuberTRoy wants to merge 1 commit intomasterfrom
feat/remove

Conversation

@HuberTRoy
Copy link
Copy Markdown
Member

@HuberTRoy HuberTRoy commented Apr 23, 2026

Summary by CodeRabbit

  • Documentation
    • Removed SubQuery Network and Miscellaneous navigation links from the documentation site.
    • Removed SubQuery Network section from sidebar navigation.
    • Removed SubQuery Network card from the homepage.
    • Removed delegators and SQT token guides from the popular guides section.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 23, 2026

📝 Walkthrough

Walkthrough

This 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

Cohort / File(s) Summary
VuePress Navigation Configuration
docs/.vuepress/config.ts, docs/.vuepress/sidebar.ts
Removed navbar links and sidebar sections for "SubQuery Network" and "Miscellaneous" documentation areas, including nested pages and collapsed sections.
Homepage Content
docs/README.md
Removed the SubQuery Network card and two related guide items (delegators and SQT token) from the homepage's popular guides list.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Poem

🐰 With a hop and a delete, we tidy the way,
Removing old paths that no longer stay,
The network and misc fade to grey,
Cleaner docs for a brighter day! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: remove network and miscellaneous' directly corresponds to the main changes, which remove SubQuery Network and Miscellaneous sections from navbar, sidebar, and homepage.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/remove

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

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 | 🟠 Major

Sidebar wiring still exposes removed sections

getSidebar("") still serves entries for Miscellaneous and Publish to the SubQuery Network (see docs/.vuepress/sidebar.ts Line 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

📥 Commits

Reviewing files that changed from the base of the PR and between bd8df2c and 56c33c5.

📒 Files selected for processing (3)
  • docs/.vuepress/config.ts
  • docs/.vuepress/sidebar.ts
  • docs/README.md
💤 Files with no reviewable changes (2)
  • docs/README.md
  • docs/.vuepress/sidebar.ts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant