-
Notifications
You must be signed in to change notification settings - Fork 0
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
bug: contract.simulate
does not accept feeCurrency
field
#11
Comments
arthurgousset
changed the title
bug:
bug: Feb 29, 2024
contract.publicWallet.simulate()
does not accept feeCurrency
fieldcontract.simulate
does not accept feeCurrency
field
This issue could be done around the same time or shortly after this: (while you have your head in viem) |
Apparently it works fine if you pass the chain explicitly to the simulate transaction params: and fails as described by @arthurgousset if you don't: For write transaction though it seems to just accept any arbitrary parameters with the chain explicitly provided: and without it as well: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is there an existing issue for this?
Package Version
2.7.9
Current Behavior
This code snippet shows an error (red squiggly line) under
feeCurrency
:The error message is:
But the same code as
contract.write.transfer
accepts thefeeCurrency
Expected Behavior
simulate
andWalletClient
calls should accept the same fields.Viem recommends that most contract calls or transactions are first simulated
Steps To Reproduce
Link to Minimal Reproducible Example (StackBlitz, CodeSandbox, GitHub repo etc.)
https://github.com/celo-org/feecurrency/blob/main/viem.ts
Anything else?
No response
Related
- celo-org/feecurrency demo repo
The text was updated successfully, but these errors were encountered: