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
update uses of messageHash and rawTransaction from eth-account to snakecase version per eth-account v0.12.2 #3348
update uses of messageHash and rawTransaction from eth-account to snakecase version per eth-account v0.12.2 #3348
Conversation
7dc38fd
to
897318d
Compare
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.
lgtm with a clarifying question. This needs to go be merged into v6, right? And v7 / main will have the new version of eth-account with the removals?
3348.deprecation.rst
Outdated
@@ -0,0 +1 @@ | |||
``eth-account`` ``messageHash`` and ``rawTransaction`` deprecated for snake_case versions |
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.
nit:
``eth-account`` ``messageHash`` and ``rawTransaction`` deprecated for snake_case versions | |
``eth-account``, ``messageHash``, [optional oxford comma] and ``rawTransaction`` deprecated for snake_case versions |
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.
Yeah, cleaned that up.
897318d
to
eebe23b
Compare
v6 has its own version getting worked. This is failing now, but I think yes, once |
We will need to make the |
…kecase version per eth-account v0.12.2, bump eth-account dep to >=0.12.2
eebe23b
to
0811a87
Compare
What was wrong?
eth-account
v0.12.2 deprecatedmessageHash
formessage_hash
andrawTransaction
forraw_transaction
.How was it fixed?
Changed all examples to use the snakecase version. In one example in
web3.eth.account.rst
, bothmessageHash
andmessage_hash
keys are present.Todo:
Cute Animal Picture