-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[Smart Accounts Kit] Add guide for Privy #2570
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
smart-accounts-kit/guides/smart-accounts/signers/embedded-wallets.md
Outdated
Show resolved
Hide resolved
m4sterbunny
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor comments, otherwise lgtm
f452bce to
19c1f12
Compare
m4sterbunny
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor style items, otherwise lgtm
Co-authored-by: m4sterbunny <[email protected]>
bgravenorst
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Description
Add Privy as a signer guide
Issue(s) fixed
Part of #2424
Preview
Checklist
External contributor checklist
Note
Adds Privy signer guide and a signers index, links them in the sidebar, and tightens delegation scope requirements; includes minor copy/config tweaks to Dynamic and Embedded Wallets guides.
guides/smart-accounts/signers/privy.md— how to use Privy with MetaMask Smart Accounts.guides/smart-accounts/signers/index.md— overview page linking Embedded Wallets, Dynamic, and Privy signer guides.guides/smart-accounts/signers/dynamic.md— minor copy fixes (context wording, connection note).guides/smart-accounts/signers/embedded-wallets.md— minor copy updates, add correctWagmiProviderimport note, and placeholder config values.gator-sidebar.jsto add a category link toguides/smart-accounts/signers/indexand includeguides/smart-accounts/signers/privyin items.guides/delegation/use-delegation-scopes/index.md(and versioned doc), change guidance from optional to required: "you must configure a scope."Written by Cursor Bugbot for commit 94c7666. This will update automatically on new commits. Configure here.