Skip to content

Commit

Permalink
Remove redundant comment about empty transaction output lists.
Browse files Browse the repository at this point in the history
It's now been established that having an empty list of outputs is
perfectly reasonable. An example of this is a transaction that includes
just a delegation certificate.
  • Loading branch information
jonathanknowles committed Oct 26, 2020
1 parent 19a0bf6 commit ed61124
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions lib/core/src/Cardano/Wallet/Api/Types.hs
Expand Up @@ -649,9 +649,6 @@ data ApiTransaction (n :: NetworkDiscriminant) = ApiTransaction
, depth :: !(Maybe (Quantity "block" Natural))
, direction :: !(ApiT Direction)
, inputs :: ![ApiTxInput n]
-- TODO: Investigate whether the list of outputs should be non-empty, and
-- if so, whether the 'outputs' field can be encoded as a non-empty list.
-- See: https://jira.iohk.io/browse/ADP-400
, outputs :: ![AddressAmount (ApiT Address, Proxy n)]
, withdrawals :: ![ApiWithdrawal n]
, status :: !(ApiT TxStatus)
Expand Down

0 comments on commit ed61124

Please sign in to comment.