Skip to content

Conversation

@AyushBherwani1998
Copy link
Member

@AyushBherwani1998 AyushBherwani1998 commented Oct 23, 2025

Description

Currently, user need to be upgraded MetaMask smart account in order to use the ERC-7715. Adds an additional step for dApp to handle the upgrade.

Issue(s) fixed

Fixes #

Preview

Checklist

  • If this PR updates or adds documentation content that changes or adds technical meaning, it has received an approval from an engineer or DevRel from the relevant team.
  • If this PR updates or adds documentation content, it has received an approval from a technical writer.

Note

Adds a new step to verify/upgrade to a MetaMask Smart Account before requesting ERC-7715 permissions, with code sample, and renumbers subsequent steps.

  • Docs — ERC-7715 guide (delegation-toolkit/guides/erc7715/execute-on-metamask-users-behalf.md, gator_versioned_docs/version-0.13.0/guides/erc7715/execute-on-metamask-users-behalf.md):
    • New Step 4: Check EOA account code
      • Explain Smart Account upgrade requirement and why (ERC-7710 delegation).
      • Add code sample to read publicClient.getCode, derive delegatorAddress (EIP-7702), compare with getDeleGatorEnvironment(...).implementations.EIP7702StatelessDeleGatorImpl to detect upgrade.
      • Provide guidance to programmatically upgrade or ask user to switch.
    • Renumber steps: "Request ERC-7715 permissions" → 5, "Set up a Viem client" → 6, "Redeem ERC-7715 permissions" → 7.

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

@AyushBherwani1998 AyushBherwani1998 requested review from a team as code owners October 23, 2025 22:04
@vercel
Copy link

vercel bot commented Oct 23, 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 Oct 27, 2025 5:43pm

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

Copy link
Contributor

@alexandratran alexandratran left a comment

Choose a reason for hiding this comment

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

Added some edits and one remaining question.

@AyushBherwani1998
Copy link
Member Author

@bgravenorst can we get your review?

Copy link
Contributor

@bgravenorst bgravenorst left a comment

Choose a reason for hiding this comment

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

lgtm

@alexandratran alexandratran merged commit 394811c into main Oct 27, 2025
15 of 18 checks passed
@alexandratran alexandratran deleted the 7715-guide-hotfix branch October 27, 2025 20:49
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