-
Notifications
You must be signed in to change notification settings - Fork 77
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
rpc: fix marshalling of type-specific tx data #789
Conversation
Codecov Report
@@ Coverage Diff @@
## master #789 +/- ##
==========================================
+ Coverage 66.8% 66.98% +0.18%
==========================================
Files 141 141
Lines 12980 13164 +184
==========================================
+ Hits 8671 8818 +147
- Misses 3899 3930 +31
- Partials 410 416 +6
Continue to review full report at Codecov.
|
ab1cc6f
to
adf370a
Compare
Fixed double-pointer bug in following methods: - GetRawTransactionVerbose - GetBlockVerbose - GetBlockHeader - GetTxOut - ValidateAddress
adf370a
to
3e27eeb
Compare
pkg/rpc/response/result/block.go
Outdated
NetFee util.Fixed8 `json:"net_fee"` | ||
|
||
Nonce uint32 `json:"nonce,omitempty"` | ||
Transaction *transaction.Transaction |
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.
Not sure it's a good change, we're missing original TxID
and Size
field, although it's possible to get them via other means. @im-kulikov: are you using any of these fields directly?
3db2ba3
to
871b181
Compare
871b181
to
9c09ad9
Compare
closes #585