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

feat(ethers-contract): typed txs (part 4) #362

Merged
merged 7 commits into from
Aug 9, 2021
Merged

Conversation

gakonst
Copy link
Owner

@gakonst gakonst commented Aug 2, 2021

  • Adds TypedTransaction inside the ContractCall object
  • Uses EIP1559 Transactions by default (Legacy if legacy feature flag is on), allows falling back to legacy ones via the legacy helper function
  • Most tests need to use legacy() because ganache does not support Berlin or London yet (?!?)

@gakonst gakonst changed the title feat(ethers-contract): typed txs feat(ethers-contract): typed txs (part 4) Aug 2, 2021
Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, minor nits.

ethers-contract/src/call.rs Outdated Show resolved Hide resolved
ethers-contract/src/factory.rs Outdated Show resolved Hide resolved
@aidenbenner
Copy link

Thanks for the amazing work on this library!
Is this ready to merge? This would be very useful now that the London fork has landed.

@gakonst
Copy link
Owner Author

gakonst commented Aug 8, 2021

I'll be working on the final bits to merge these PRs -today-, there's currently a serialization bug (along with some extra tests against testnets)

Base automatically changed from feat/typed-txs-rpc to master August 9, 2021 00:31
@gakonst gakonst merged commit 25c2e0e into master Aug 9, 2021
@gakonst gakonst deleted the feat/typed-txs-contracts branch August 9, 2021 00:50
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

3 participants