-
Notifications
You must be signed in to change notification settings - Fork 110
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
feat: Add onchain kit paymaster utilities #515
Conversation
abbc55b
to
1aaacd9
Compare
9513af6
to
996459a
Compare
export const client = createPublicClient({ | ||
chain: baseSepolia, | ||
transport: http(), | ||
}); |
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.
This is used as the client to pass into isWalletASmartWallet
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.
I believe we already have a client, no?
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.
Had to use client
rather than paymasterClient
because client
is created using createPublicClient
and is of type PublicClient
, which is what isWalletASmartWallet
expects.
Maybe we should update isWalletASmartWallet
to use createClient
rather than createPublicClient
from Viem?
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.
I think that isWalletASmartWallet
should continue to use createClient
because it is a "public" endpoint that doesn't require authentication to use.
37cc1f4
to
8bb2c1e
Compare
8bb2c1e
to
7464c05
Compare
e51889a
to
f00d066
Compare
What changed? Why?
Add
isValidAAEntrypoint
andisWalletASmartWallet
OnchainKit utilities to thepaymaster proxy
.isValidAAEntrypoint
https://onchainkit.xyz/wallet/is-valid-aa-entrypointisWalletASmartWallet
https://onchainkit.xyz/wallet/is-wallet-a-smart-walletUpdate and add unit tests.
Notes to reviewers
How has it been tested?
Tested locally and in Dev.