-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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.get_balance, eth.get_code, eth.get_transaction_count #2056
Conversation
a1a7c43
to
5046361
Compare
@@ -40,8 +40,6 @@ | |||
from web3 import Web3 # noqa: F401 | |||
from web3.module import Module # noqa: F401 | |||
|
|||
Munger = Callable[..., Any] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this change just to simplify it a bit since this is pretty general anyhow?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops, I don't remember why I did this. Will put it back!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kclowes LGTM!
e625da0
to
b7e5f3d
Compare
my code w3.eth.get_code(account.address)) return b'' (empty) |
What was wrong?
Added async
eth.get_balance
,eth.get_code
, andeth.get_transaction_count
(all methods that use theblock_id_munger
).Related to Issue #1413
Todo:
Cute Animal Picture