balanced / balanced-api

Balanced API specification.
220 stars 72 forks source link

Return the bank account removal reason via noc_reason instead of meta #733

Open kyungmin opened 9 years ago

kyungmin commented 9 years ago

Currently bank account removal reason shows up in meta['balanced.noc_reason'] in bank accounts. This can potentially edited or removed by users. This should show up in a new field, like noc_reason.

Reference: https://github.com/balanced/balanced-dashboard/issues/1619

mjallday commented 9 years ago

noc_reason is going to be useless on the majority of bank accounts. why not fire an event (e.g. bank_account.invalidated) and include some meta information in that event?