-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[Smart Accounts Kit] Update docs for Advanced Permissions doc #2491
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/advanced-permissions/execute-on-metamask-users-behalf.md
Outdated
Show resolved
Hide resolved
smart-accounts-kit/guides/advanced-permissions/execute-on-metamask-users-behalf.md
Outdated
Show resolved
Hide resolved
smart-accounts-kit/guides/advanced-permissions/execute-on-metamask-users-behalf.md
Outdated
Show resolved
Hide resolved
smart-accounts-kit/guides/advanced-permissions/execute-on-metamask-users-behalf.md
Outdated
Show resolved
Hide resolved
smart-accounts-kit/guides/advanced-permissions/use-permissions/erc20-token.md
Outdated
Show resolved
Hide resolved
smart-accounts-kit/guides/advanced-permissions/use-permissions/erc20-token.md
Outdated
Show resolved
Hide resolved
smart-accounts-kit/guides/advanced-permissions/use-permissions/native-token.md
Outdated
Show resolved
Hide resolved
smart-accounts-kit/guides/advanced-permissions/use-permissions/native-token.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 nits to fix
Co-authored-by: m4sterbunny <m4sterbunny@gmail.com>
alexandratran
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 edits, LGTM.
smart-accounts-kit/guides/advanced-permissions/use-permissions/erc20-token.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.
Consensus not achieved around verbose link offering; deferring to wider team for decision.
Description
Issue(s) fixed
Fixes #
Preview
Checklist
External contributor checklist
Note
Clarifies Advanced Permissions behavior (auto-upgrade in Flask >=13.9.0) and adds concise API reference links for requesting/redeeming permissions and ERC-20/native token permission types.
13.9.0; note manual/programmatic upgrade for earlier versions.requestExecutionPermissionsand add link to its API reference.sendUserOperationWithDelegation,sendTransactionWithDelegation) and add links to their API references.Written by Cursor Bugbot for commit 78cc3e8. This will update automatically on new commits. Configure here.