Skip to content
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

Outdated recovery example #10

Open
ennioVisco opened this issue Jun 3, 2024 · 2 comments
Open

Outdated recovery example #10

ennioVisco opened this issue Jun 3, 2024 · 2 comments

Comments

@ennioVisco
Copy link

We noted two inconsistencies that are very frustrating:

  1. The doc is not updated with the current code, more precisely the recovery information is passed differently here and here.
  2. Moreover, in both cases, we cannot seem to make it work because we keep getting any/the first userOp reverted with 0x756688fe (which seems to be "InvalidNonce"). This holds true even if the nonce is passed by calling kernelAccount.getNonce(nonceKey).

Any ideas how to solve it? We love the toolkit you people at ZeroDev are building, and we'd really appreciate if you supported us in getting started with them!

@ennioVisco
Copy link
Author

@SahilVasava any updates about this? It is fairly blocking for us in adopting ZeroDev in production...

@ennioVisco
Copy link
Author

ennioVisco commented Jul 9, 2024

@SahilVasava something is off about the updated example:

const kernelClient = createKernelAccountClient({

The client that executes the recovery action is created from the account corresponding to the sudo validator, but it should be the one corresponding to the regular one, otherwise this does not make any sense (if a recovery action is started, presumably, it's because the access to the sudo validator is not available...).

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

No branches or pull requests

1 participant