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

Async eth.send_raw_transaction #2135

Merged
merged 3 commits into from
Sep 7, 2021
Merged

Async eth.send_raw_transaction #2135

merged 3 commits into from
Sep 7, 2021

Conversation

ediskandarov
Copy link
Contributor

@ediskandarov ediskandarov commented Sep 7, 2021

What was wrong?

Added async eth.send_raw_transaction.

Related to Issue #1413

How was it fixed?

Todo:

Cute Animal Picture

lionel-animals-to-follow-on-instagram-1568319926

@ediskandarov ediskandarov changed the title [WIP] Async eth.send_raw_transaction Async eth.send_raw_transaction Sep 7, 2021
@ediskandarov
Copy link
Contributor Author

@kclowes please review

@fselmo fselmo force-pushed the master branch 2 times, most recently from f86475a to 4b658dd Compare September 7, 2021 18:03
@fselmo fselmo merged commit 3fb3a3e into ethereum:master Sep 7, 2021
@fselmo
Copy link
Collaborator

fselmo commented Sep 7, 2021

Thanks @emcpow2! I added a test for this and split the newsfragments up into a feature and a doc, assessed some errors with the test, squashed, and merged!

We appreciate the contribution!

@ediskandarov
Copy link
Contributor Author

Thanks @fselmo !

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.

2 participants