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

Added stateOverride to estimateGas action #2242

Closed
wants to merge 1 commit into from

Conversation

conwayconstar
Copy link
Contributor

@conwayconstar conwayconstar commented May 10, 2024

This PR adds stateOverride to estimateGas action as discussed here #2234


PR-Codex overview

The focus of this PR is to implement the stateOverride feature on the estimateGas action in the codebase.

Detailed summary

  • Added stateOverride parameter to estimateGas action
  • Updated WalletRpcSchema type in eip1193.ts
  • Added stateOverride handling in estimateGas function
  • Imported parseStateOverride and StateOverride types

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

changeset-bot bot commented May 10, 2024

🦋 Changeset detected

Latest commit: e62b65b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
viem Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented May 10, 2024

@conwayconstar is attempting to deploy a commit to the wevm Team on Vercel.

A member of the Team first needs to authorize it.

@jxom
Copy link
Member

jxom commented May 19, 2024

Thanks! Superseded by #2275. Your commits are preserved.

@jxom jxom closed this May 19, 2024
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.

None yet

2 participants