Skip to content

Comments

docs: remove walletkit pages and add migration guide#740

Merged
Sam-Newman merged 1 commit intomainfrom
devin/1770735160-walletkit-migration
Feb 12, 2026
Merged

docs: remove walletkit pages and add migration guide#740
Sam-Newman merged 1 commit intomainfrom
devin/1770735160-walletkit-migration

Conversation

@devin-ai-integration
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Feb 10, 2026

Description

Removes five WalletKit-related pages from Reown Docs whose content already exists on the WalletConnect Docs, and replaces the walletconnect-deprecations nav entry with a new WalletKit Migration page that redirects readers to the corresponding WalletConnect Docs locations.

Deleted pages:

Added page:

  • advanced/walletkit-migration.mdx — Table mapping each removed page to its WalletConnect Docs equivalent

Navigation changes (docs.json):

  • Removed cloud/explorer-submission from the WalletGuide group
  • Removed advanced/api/sign/wallet-usage from the Sign API group
  • Replaced advanced/walletconnect-deprecations with advanced/walletkit-migration

Note: walletconnectmodal/usage.mdx and walletconnectmodal/resources.mdx had no navigation entries in docs.json, so only the files were removed.

Human Review Checklist

  • Verify the WalletGuide nav section renders correctly — explorer-submission was the first item; the group now starts with nested "WalletGuide Chains" subgroup
  • Confirm no internal links in other Reown docs pages reference the five deleted files (especially cloud/explorer-submission and advanced/api/sign/wallet-usage which were navigable)
  • Verify all external links in the migration page resolve correctly on the WalletConnect Docs site

Requested by: Jem Conlon
Link to Devin run

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

Co-Authored-By: Jem Conlon <jem@walletconnect.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

@github-actions
Copy link

@devin-ai-integration[bot] Please review the tone of voice for the content changes in this PR against Reown's brand guidelines.

📝 Content Review Request

Files to review: 1

  • advanced/walletkit-migration.mdx

Review focus:

  • Tone alignment with Reown's brand guidelines
  • Clear and accessible language for developers
  • Professional yet approachable communication
  • Consistent terminology usage

Guidelines summary:

  • Clear & Accessible: Translate complex ideas into approachable language
  • Professional yet Friendly: Maintain authority while being welcoming
  • Developer-Focused: Understand technical audience but remain inclusive
  • Avoid: Overly casual language, fear-based messaging, buzzwords, jargon without explanation
  • Embrace: Clear explanations, confident tone, transparency, respectful communication

Please analyze the content changes and provide constructive feedback on tone and voice alignment.

@Sam-Newman Sam-Newman merged commit 85c6a91 into main Feb 12, 2026
5 checks passed
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