Skip to content

Fix: remove signerMeta from the advanced permissions starter#86

Open
mj-kiwi wants to merge 1 commit intomainfrom
fix/remove-signer-meta
Open

Fix: remove signerMeta from the advanced permissions starter#86
mj-kiwi wants to merge 1 commit intomainfrom
fix/remove-signer-meta

Conversation

@mj-kiwi
Copy link

@mj-kiwi mj-kiwi commented Mar 2, 2026

What?

  • remove the signerMeta from the advanced permissions starter

Why?

  • no signerMeta is return from the grant permission method

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 RedeemPermissionButton to stop reading delegation data from permission.signerMeta and instead use permission.context and permission.delegationManager directly.

This removes the early error path for missing signerMeta and keeps validation focused on the required context/delegationManager fields before sending the delegated user operation.

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

@mj-kiwi mj-kiwi requested review from a team as code owners March 2, 2026 04:25
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.

1 participant