Skip to content

Conversation

@Ansonhkg
Copy link
Collaborator

WHAT

The viewPKPPermissions method throws accountOrWalletClient is required but was not provided when using an address as the PKP identifier, but works correctly with tokenId or pubkey identifiers.

In getPermissionsContext.ts, the call to resolvePkpTokenId() is missing the required third parameter accountOrWalletClient:

// Current (broken)
const tokenId = (await resolvePkpTokenId(identifier, networkCtx)).toString();

// Fixed
const tokenId = (await resolvePkpTokenId(identifier, networkCtx, accountOrWalletClient)).toString();

@Ansonhkg Ansonhkg merged commit 345a78c into feat/naga-dev-29-05-2025-v2 Jul 25, 2025
1 of 4 checks passed
@Ansonhkg Ansonhkg deleted the feature/node-4565-error-accountorwalletclient-is-required-but-was-not-provided branch July 25, 2025 22:41
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.

2 participants