Skip to content

Conversation

@yihuang
Copy link
Contributor

@yihuang yihuang commented Jul 21, 2025

Description

for eip-7702 tx type, estimate gas lose field SetCodeAuthorizations when modify the message.

Closes: #363


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • tackled an existing issue or discussed with a team member
  • left instructions on how to review the changes
  • targeted the main branch

Reviewers Checklist

All items are required.
Please add a note if the item is not applicable
and please add your handle next to the items reviewed
if you only reviewed selected items.

I have...

  • added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • confirmed all author checklist items have been addressed
  • confirmed that this PR does not change production code
  • reviewed content
  • tested instructions (if applicable)
  • confirmed all CI checks have passed

@yihuang yihuang requested review from a team as code owners July 21, 2025 09:08
@aljo242
Copy link
Contributor

aljo242 commented Jul 21, 2025

@yihuang can you please link an existing github issue for this? If one does not exist, feel free to create one. We want to ensure that all PRs have an associated issue for context.

@aljo242
Copy link
Contributor

aljo242 commented Jul 28, 2025

@yihuang bump on this

1 similar comment
@aljo242
Copy link
Contributor

aljo242 commented Jul 28, 2025

@yihuang bump on this

@yihuang
Copy link
Contributor Author

yihuang commented Jul 28, 2025

@yihuang bump on this

#363 issue created

@aljo242 aljo242 self-assigned this Jul 28, 2025
@aljo242 aljo242 enabled auto-merge July 28, 2025 16:39
@aljo242 aljo242 added this pull request to the merge queue Jul 28, 2025
Merged via the queue into cosmos:main with commit bfa2f05 Jul 28, 2025
16 checks passed
zsystm pushed a commit to zsystm/evm that referenced this pull request Nov 2, 2025
Co-authored-by: Alex | Interchain Labs <alex@interchainlabs.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: estimateGas missing message field

2 participants