Fix: remove signerMeta from the advanced permissions starter#86
Conversation
…ermission handling
jeffsmale90
left a comment
There was a problem hiding this comment.
This looks good to me. @AyushBherwani1998 any context that we are missing here?
|
I was waiting for the monorepo to module template to get merged. As changing templates will bring in merge conflicts. For this to reflect in the CLI, we need to publish new version which is blocked by migration of monorepo PR. I'll resolve conflicts on my PR. |
|
@mj-kiwi @jeffsmale90 we should revert the PR. The templates are targeting the latest stable version 0.3.0, which doesn't follow the Ideally this should be done when updating templates and examples to next stable version release. |
|
The problem is arising because latest Flask release is not compatible with 0.3.0, given the changes done according to new ERC-7715 spec. We can release a new version |
What?
Why?
How?
Note
Low Risk
Low risk template fix that only changes how delegation parameters are read/validated; behavior should be unchanged aside from no longer failing on missing
signerMeta.Overview
Updates the advanced permissions starter’s
RedeemPermissionButtonto stop reading delegation data frompermission.signerMetaand instead usepermission.contextandpermission.delegationManagerdirectly.This removes the early error path for missing
signerMetaand keeps validation focused on the requiredcontext/delegationManagerfields before sending the delegated user operation.Written by Cursor Bugbot for commit 5be4caf. This will update automatically on new commits. Configure here.