Branch: refs/heads/master
Home: https://github.com/btcsuite/btcwallet
Commit: fb06a6bd046dbd9a981f6bd2c65bf0658a968218
https://github.com/btcsuite/btcwallet/commit/fb06a6bd046dbd9a981f6bd2c65bf0658a968218
Author: Josh Rickmar <[email protected]>
Date: 2016-04-28 (Thu, 28 Apr 2016)
Changed paths:
M rpc/api.proto
M rpc/documentation/api.md
M rpc/rpcserver/server.go
M rpc/walletrpc/api.pb.go
M wallet/notifications.go
Log Message:
-----------
Cleanup TransactionDetails RPC messages.
Remove the addresses field from TransactionDetails.Output. It is
assumed that the caller is able to deserialize the transaction and
encode the output scripts to addresses, so this is unnecessary server
overhead and conflicts with the current API philosophy of not
duplicating data already included in another field.
Since there is no additional data included for outputs not controlled
by the wallet, remove the `mine` specifier from the Output message and
replace it with an output index. Only include messages for controlled
outputs, rather than creating messages for both controlled and
uncontrolled outputs. Rename the repeated field from `outputs` to
`credits` to be consistent with the `debits` field.
Bump major API version as this is a breaking change.
Closes #408.
