-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[Delegation Toolkit][v0.13] Update creating delegation references #2282
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
* 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>
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
Can merge once comments are addressed
delegation-toolkit/guides/delegation/execute-on-smart-accounts-behalf.md
Outdated
Show resolved
Hide resolved
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, |
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.
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.
Description
Issue(s) fixed
Fixes #2242
Preview
Checklist