Skip to content

Conversation

AyushBherwani1998
Copy link
Member

Description

  • Update createDelegation/ createOpenDelegation references to include scope, and other parameters introduced in v0.13.

Issue(s) fixed

Fixes #2242

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.

alexandratran and others added 6 commits September 5, 2025 10:23
* Minor DTK fixes

* Apply suggestions from code review

Co-authored-by: Ayush Bherwani <ayush.bherwani1998@gmail.com>

---------

Co-authored-by: Ayush Bherwani <ayush.bherwani1998@gmail.com>
@AyushBherwani1998 AyushBherwani1998 requested review from a team as code owners September 8, 2025 13:32
@AyushBherwani1998 AyushBherwani1998 changed the base branch from main to feat/v0.13-dev September 8, 2025 13:33
Copy link

vercel bot commented Sep 8, 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 Sep 8, 2025 6:29pm

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.

Can merge once comments are addressed

Co-authored-by: Alexandra Carrillo <12214231+alexandratran@users.noreply.github.com>
@@ -32,6 +32,7 @@ const delegation = createDelegation({
},
to: delegateAccount,
from: delegatorAccount,
environment: delegatorAccount.environment,
Copy link

Choose a reason for hiding this comment

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

Bug: Incorrect Smart Account Property Access

The createDelegation examples incorrectly access delegatorAccount.environment. The environment property is specific to smart accounts, but delegatorAccount is typically an EOA and lacks this property. This pattern appears in several scope examples throughout the guides.

Additional Locations (2)

Fix in Cursor Fix in Web

@AyushBherwani1998 AyushBherwani1998 merged commit d0e365d into feat/v0.13-dev Sep 8, 2025
5 checks passed
@AyushBherwani1998 AyushBherwani1998 deleted the feat/guide-improvements branch September 8, 2025 18:30
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.

Add guides for using delegation scopes
2 participants