View Issue Details

IDProjectCategoryView StatusLast Update
0007275Talerspecificationpublic2024-01-12 14:03
ReporterChristian Grothoff Assigned ToChristian Grothoff  
PriorityimmediateSeveritytweakReproducibilityN/A
Status closedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Target Version0.9.3Fixed in Version0.9.3 
Summary0007275: debit_account / credit_account are highly redundant in bank-api transaction histories
DescriptionWhen requesting a credit/debit history, the affected account is repeated in every single entry of the account history.
This wastes bandwidth and computation time (parsing!) and is simply inelegant. We should refactor the API to only return the main account the credit/debit histories are for ONCE (or zero? after all, the client should know what it requested?), and certainly not for every transaction.

Note: this change needs to be coordinated across all bank clients and implementations.
TagsNo tags attached.

Relationships

related to 0007300 closedFlorian Dold Taler bank access API does not follow conventions 

Activities

Christian Grothoff

2023-09-03 17:36

manager   ~0020485

Let's fix this together with 0007300.

Christian Grothoff

2023-09-03 17:55

manager   ~0020486

Fixed in docs.git.

Christian Grothoff

2023-09-03 17:59

manager   ~0020487

5adf8fab..d289bb87 changes merchant.git (Anastasis/merchant facade) to use the non-redundant API style.

Christian Grothoff

2023-09-03 18:00

manager   ~0020488

exchange.git::f7de830b..fd6a1d1a modifies the exchange bank and fakebank to use the new non-redundant style.

MS

2023-09-05 09:35

reporter   ~0020494

libeufin.git at 7ccac24e29bd825ac2e0e721226ccabde6d947cc got changed after this bug. Setting as feedback until tests from the exchange code-base confirm the correctness.

Christian Grothoff

2023-09-05 14:58

manager   ~0020502

Can't yet confirm, as all tests were migrated to fakebank and/or are commented out because they were failing for a while. Will investigate.

Christian Grothoff

2023-09-23 14:59

manager   ~0020524

Solved at least in fakebank, libeufin will have it fixed once 0.9.3 is out.

Issue History

Date Modified Username Field Change
2022-07-05 13:12 Christian Grothoff New Issue
2022-07-05 13:12 Christian Grothoff Status new => assigned
2022-07-05 13:12 Christian Grothoff Assigned To => Christian Grothoff
2022-07-05 14:55 Christian Grothoff Assigned To Christian Grothoff => MS
2022-11-02 15:32 Christian Grothoff Priority urgent => normal
2022-12-20 19:38 Christian Grothoff Target Version => 0.11
2023-09-03 17:35 Christian Grothoff Target Version 0.11 => 0.9.3
2023-09-03 17:36 Christian Grothoff Note Added: 0020485
2023-09-03 17:36 Christian Grothoff Relationship added related to 0007300
2023-09-03 17:55 Christian Grothoff Note Added: 0020486
2023-09-03 17:59 Christian Grothoff Note Added: 0020487
2023-09-03 18:00 Christian Grothoff Note Added: 0020488
2023-09-03 18:00 Christian Grothoff Priority normal => immediate
2023-09-05 09:35 MS Note Added: 0020494
2023-09-05 09:35 MS Status assigned => feedback
2023-09-05 14:58 Christian Grothoff Assigned To MS => Christian Grothoff
2023-09-05 14:58 Christian Grothoff Note Added: 0020502
2023-09-23 14:59 Christian Grothoff Status feedback => resolved
2023-09-23 14:59 Christian Grothoff Resolution open => fixed
2023-09-23 14:59 Christian Grothoff Fixed in Version => 0.9.3
2023-09-23 14:59 Christian Grothoff Note Added: 0020524
2023-09-23 15:09 Christian Grothoff Status resolved => closed
2024-01-12 14:03 Christian Grothoff Category exchange API (HTTP specification) => specification