balanced / balanced-api

Balanced API specification.
220 stars 72 forks source link

Require settlement link on Credit resource #737

Closed remear closed 9 years ago

mjallday commented 9 years ago

@remear how would i see all settlements past and present for this particular transaction? does the same thing already exist for reversals?

remear commented 9 years ago

The ability to view all settlements for a transaction does not exist yet as it seems the concern was only the latest settlement. I do feel this might be useful, in which case it would make sense to add a settlements link that returns all settlements that concerned the transaction.

A reversal does not have a settlement link. I'd enjoy some discussion on why it may or may not need one.

mjallday commented 9 years ago

A reversal does not have a settlement link

I think it would require one because a reversal against a previously settled credit will belong to a settlement at some point. You cannot infer the reversal's settlement from the credit since it could happen in the same settlement but could also be associated to a different settlement.