Skip to content

docs: remove WalletGuide section from Advanced tab#748

Open
devin-ai-integration[bot] wants to merge 1 commit intomainfrom
devin/1771850935-remove-walletguide-section
Open

docs: remove WalletGuide section from Advanced tab#748
devin-ai-integration[bot] wants to merge 1 commit intomainfrom
devin/1771850935-remove-walletguide-section

Conversation

@devin-ai-integration
Copy link
Contributor

Description

Removes the WalletGuide navigation group from the Advanced tab sidebar in docs.json. This group contained:

  • WalletGuide Chains (cloud/chains/overview, cloud/chains/chain-list)
  • WalletGuide Wallets (cloud/wallets/wallet-list)
  • Cloud Explorer (cloud/explorer)

Note: The underlying .mdx content pages are not deleted — they are only removed from the sidebar navigation. If these pages should also be removed, a follow-up change would be needed.

⚠️ Reviewer attention: Several other docs pages still reference WalletGuide (e.g., installation guides, options pages). Confirm whether those inline references should also be updated or if only the sidebar entry needed removal.

Link to Devin run: https://app.devin.ai/sessions/ab60b295f0014577b96b82195c0e6501
Requested by: TomTom

Tests

  • Ran the changes locally with Mintlify and confirmed that the changes appear as expected.
  • Ran a grammar check on the updated/created content using ChatGPT.

Direct link to the deployed preview files

  • Will be available after CI preview deployment completes.

Co-Authored-By: TomTom <tomtom@reown.com>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@claude
Copy link

claude bot commented Feb 23, 2026

Claude finished @devin-ai-integration[bot]'s task in 1m 23s —— View job


PR Review

  • Read changed files and context
  • Check for broken references/links
  • Post findings
Found 1 issue(s)

Issue 1: Orphaned internal links to removed navigation pages

ID: docs-orphaned-chain-list-links-a3f1
File: appkit/networks/supported-chains.mdx:9,56
Severity: MEDIUM
Category: breaking_change

Context:

  • Pattern: supported-chains.mdx links to /cloud/chains/chain-list (lines 9 and 56) — a page that is now removed from sidebar navigation entirely
  • Risk: The page still exists on disk and will render, but has no sidebar entry. Users who follow the link land on a page with no navigation context and no way to discover it through the sidebar.
  • Impact: Confusing UX; if those .mdx files are cleaned up in a follow-up PR, these links will 404
  • Trigger: Any user clicking the "WalletConnect Cloud Supported Chains" link in the supported-chains page

Recommendation: Either keep the cloud/chains/chain-list page in the sidebar (under a different group), or update supported-chains.mdx to link to the Reown dashboard or another canonical source:

- Line 9: replace `/cloud/chains/chain-list` with a redirect target or remove the sentence
- Line 56: same — update or remove the cross-reference

Fix this →

Additional notes (non-blocking):

  • ~15 files across appkit/ and advanced/ still reference walletguide.walletconnect.network as an external URL. If this service is being deprecated (which this PR implies), those inline references should be updated in a follow-up. The PR description already flags this — just confirming the scope.
  • The redirect at docs.json:50 (/advanced/multichain/chain-listcloud/chains/chain-list) still exists and is fine since the underlying .mdx file remains on disk.
  • The change itself (14-line deletion from docs.json) is correct and does exactly what it states.

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.

0 participants