Skip to content

Conversation

@fselmo
Copy link
Collaborator

@fselmo fselmo commented Mar 26, 2024

What was wrong?

closes #3255

How was it fixed?

  • According to the spec, and our documentation, it is the dictionary itself that is passed in, not the str representation.

Todo:

Cute Animal Picture

0_o

@fselmo fselmo marked this pull request as ready for review March 26, 2024 23:20
@fselmo fselmo requested review from pacrob and reedsa March 26, 2024 23:38
Copy link
Contributor

@reedsa reedsa left a comment

Choose a reason for hiding this comment

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

Animal is very expressive, code lgtm!

Copy link
Contributor

@pacrob pacrob left a comment

Choose a reason for hiding this comment

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

:shipit:

@fselmo fselmo merged commit 6d634c6 into ethereum:main Mar 27, 2024
@fselmo fselmo deleted the fix-eth-sign-typed-data-typing branch April 3, 2024 20:49
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.

eth.sign_typed_data jsonMessage argument type discrepancy

3 participants