Skip to content

add react native v9 docs#2942

Open
yashovardhan wants to merge 1 commit into
mainfrom
react-native
Open

add react native v9 docs#2942
yashovardhan wants to merge 1 commit into
mainfrom
react-native

Conversation

@yashovardhan
Copy link
Copy Markdown
Member

@yashovardhan yashovardhan commented Jun 2, 2026

add react native v9 docs


Note

Low Risk
Documentation-only changes with no runtime or authentication logic; risk is limited to doc accuracy and broken links if navigation is not updated elsewhere.

Overview
This PR replaces the React Native Embedded Wallets docs for the v9 hooks-first SDK (Web3AuthProvider, useWeb3Auth, connectTo + AUTH_CONNECTION) and retires the old new Web3Auth / LOGIN_PROVIDER / per-version migration pages.

The getting started and advanced guides are rewritten around web3authConfig.ts, @web3auth/react-native-sdk/setup, Metro withWeb3Auth, and shared MDX partials for init options, chains, provider setup, and login params. Custom auth, MFA, dapp share, whitelabel, and smart accounts now document dashboard connection IDs, groupedAuthConnectionId, and accountAbstractionConfig instead of loginConfig and separate AA provider packages.

New content includes a consolidated Migrate to v9 guide (API map, agent prompt, checklist) and a hooks section (overview plus per-hook pages). Usage pages map v8 methods to hooks; Solana integration docs drop @web3auth/solana-provider in favor of web3Auth.signer / TransactionSigner. Incremental rn-v3–v8 migration guide files are removed in favor of the single v9 migration doc.

Reviewed by Cursor Bugbot for commit 947c5c0. Bugbot is set up for automated code reviews on this repo. Configure here.

@yashovardhan yashovardhan requested review from a team as code owners June 2, 2026 18:41
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 2, 2026

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

Project Deployment Actions Updated (UTC)
metamask-docs Ready Ready Preview, Comment Jun 2, 2026 6:41pm

Request Review

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 947c5c0. Configure here.

import { AUTH_CONNECTION, useWeb3AuthConnect } from '@web3auth/react-native-sdk'

async function loginWithFirebase(idToken: string) {
const { connectTo } = useWeb3AuthConnect()
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hooks called outside components

High Severity

Several v9 examples call useWeb3AuthConnect() or useWeb3Auth() inside plain async helpers instead of React function components or custom hooks. That pattern violates the Rules of Hooks and triggers React’s invalid hook call error if developers follow the docs.

Additional Locations (2)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 947c5c0. Configure here.


const signMessage = async (message: string) => {
const ep = new ethers.BrowserProvider(provider!)
return (await ep.getSigner()).signMessage(message)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Ethers promises not awaited

Medium Severity

The new EVM HomeView example returns (await ep.getSigner()).getAddress() and (await ep.getSigner()).signMessage(message) without awaiting those methods. In ethers v6 both return promises, so callers get a Promise instead of an address or signature string.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 947c5c0. Configure here.

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