-
Notifications
You must be signed in to change notification settings - Fork 160
Add API for compatible query of two types of transactions #683
Conversation
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.
Looks good! See the first comment for another proposed approach
5554a1f
to
9ed633d
Compare
why deploy contract test failed ? |
@fedekunze Can this pr be merged? |
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.
untested ACK
What was this supposed to fix? I'm thinking that this was a way to interpret the EthereumTxMsg that was generated from a transaction using RPC when querying the tx hash using |
Closes: #630
Description
For contributor use:
docs/
) or specification (x/<module>/spec/
)godoc
comments.Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorerFor admin use:
WIP
,R4R
,docs
, etc)