Skip to content

Conversation

@yashovardhan
Copy link
Member

@yashovardhan yashovardhan commented Dec 3, 2025

Multiple doc links with "./" were erroring out in production, link the ones in https://docs.metamask.io/embedded-wallets/sdk/react#advanced-configuration

This PR replaces them with direct links to the particular page/ section. For the common component, I have removed and directly copied the content to the respective file.


Note

Replaces relative links with explicit .mdx paths across Embedded Wallets docs, inlines advanced configuration content into SDK pages, and updates the Arbitrum JSON‑RPC reference link.

  • Documentation
    • Replace relative links with explicit .mdx paths across dashboard and SDK docs (Android, iOS, Flutter, React, Vue, React Native, Unity, Unreal, JS), including migration guides and troubleshooting references.
    • Remove shared embedded-wallets/sdk/_common/_advanced-config-options.mdx and inline its content into js/react/vue advanced configuration pages.
  • References
    • Update Arbitrum JSON-RPC "differences" link to the new official docs URL.

Written by Cursor Bugbot for commit ce831d6. This will update automatically on new commits. Configure here.

@yashovardhan yashovardhan requested review from a team as code owners December 3, 2025 17:48
@vercel
Copy link

vercel bot commented Dec 3, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
metamask-docs Ready Ready Preview Comment Dec 8, 2025 6:09am

bgravenorst
bgravenorst previously approved these changes Dec 3, 2025
Signed-off-by: bgravenorst <byron.gravenorst@consensys.net>
Signed-off-by: bgravenorst <byron.gravenorst@consensys.net>
@bgravenorst
Copy link
Contributor

@yashovardhan, I converted those links from absolute to relative. Our link checker handles relative links better, and it's a bit easier for us to maintain. The documentation is currently inconsistent regarding how we link internally. Let me know if you're ok with this. cc @m4sterbunny

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.

4 participants