Skip to content

docs(wallets): fix stale OTP params, deprecated useGetUserEmail refs, magic-link keys#78

Merged
SahilVasava merged 2 commits into
srinjoy/update-wallet-docsfrom
sahil/docs-wallet-fixes
Jul 2, 2026
Merged

docs(wallets): fix stale OTP params, deprecated useGetUserEmail refs, magic-link keys#78
SahilVasava merged 2 commits into
srinjoy/update-wallet-docsfrom
sahil/docs-wallet-fixes

Conversation

@SahilVasava

Copy link
Copy Markdown
Contributor

Stacked on top of #76 (srinjoy/update-wallet-docs) — base is that branch, not main.

Small, decision-free accuracy fixes found while reviewing #76. Verified against the SDK source.

Changes

  • use-send-otp.mdx — remove the emailCustomization / magicLinkTemplate parameter (it no longer exists in the SDK; sendOTP takes { email } only), and add the otpEncryptionTargetBundle return value that the hook actually returns (the OTP guide already uses it, and useVerifyOTP requires it). Return type now matches use-send-magic-link.mdx.
  • email-otp.mdx, google-oauth.mdx — the inline "email is available through useGetUserEmail" bullets pointed at a deprecated hook (per the review thread on the hook page). Reworded to useAuthenticators as inline code (no link, since that reference page isn't added yet — andras is adding it).
  • magic-link.mdx — align the sessionStorage keys (zerodevMagicLinkOtpId/zerodevMagicLinkBundlemagicLinkOtpId/magicLinkBundle) so they match use-send-magic-link.mdx and use-verify-magic-link.mdx.

Deliberately excluded (need a decision, not mechanical)

@vercel

vercel Bot commented Jul 2, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Jul 2, 2026 3:54pm

Request Review

@SahilVasava SahilVasava mentioned this pull request Jul 2, 2026
@SahilVasava SahilVasava merged commit 2980320 into srinjoy/update-wallet-docs Jul 2, 2026
1 of 2 checks passed
@SahilVasava SahilVasava deleted the sahil/docs-wallet-fixes branch July 2, 2026 15:54
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