-
Notifications
You must be signed in to change notification settings - Fork 73
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
User story: Execute userOp with Passkey and paymaster #357
Conversation
b60675d
to
5d9e53f
Compare
I noticed that we had 2 user story directories and that the file naming was inconsistent. I fixed it here: #363 , but this PR does not follow the "standard" either, can you fix it as well? |
34fcef7
to
827b321
Compare
Pull Request Test Coverage Report for Build 8649185035Details
💛 - Coveralls |
Co-authored-by: Shebin John <admin@remedcu.com>
Co-authored-by: Shebin John <admin@remedcu.com>
…re-276-user-story-paymaster
Co-authored-by: Mikhail <16622558+mmv08@users.noreply.github.com>
Co-authored-by: Nicholas Rodrigues Lordello <nick@safe.global>
Co-authored-by: Nicholas Rodrigues Lordello <nick@safe.global>
be4ccf5
to
af92a4a
Compare
const publicKey = decodePublicKey(credential.response) | ||
await signerFactory.createSigner(publicKey.x, publicKey.y, verifier.target) | ||
const signer = await signerFactory.getSigner(publicKey.x, publicKey.y, verifier.target) | ||
|
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.
seems like both tests deploy a signer right after the setup, can it be a part of the general setup?
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.
Agree it might be nice (so the tests assume an existing signer).
Approved to reduce back and forth, but agree with #357 (comment) |
Fixes #276
Todo:
Evaluate if
TestPaymaster
contract is needed with validation logic. If not needed, replace with aMockContract
instanceChanges in PR: