View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0008374 | Taler | wallet-core | public | 2024-02-13 02:38 | 2024-03-05 23:43 |
| Reporter | avalos | Assigned To | Florian Dold | ||
| Priority | normal | Severity | feature | Reproducibility | always |
| Status | closed | Resolution | won't fix | ||
| Platform | arm64 | OS | LineageOS | OS Version | 18.1 |
| Product Version | git (master) | ||||
| Target Version | 0.10 | ||||
| Summary | 0008374: Add currencySpecification to getBalances response | ||||
| Description | Currently, in order to get the currencySpecification of all the balances in the wallet, I need to make a separate getCurrencySpecification request for each balance (using its scopeInfo as an argument). This is not the most optimal approach. wallet-core already knows the relationship between balance -> scopeInfo -> currencySpecification, so why not give it to us in advance? :) | ||||
| Tags | No tags attached. | ||||
|
|
We should keep each API call simple and specific, and there could even be cases (like wallet-cli) where we don't even want to show the currency specs. I think it is fine to require dedicated calls when a currency spec is needed. Keeps the individual calls simple instead of returning possibly completely unnecessary data. For example, you may also call getBalances *a lot* (whenever the balance changes), but the currency specs the UIs probably should _cache_ as they are highly unlikely to change during the GUI's runtime :-). |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2024-02-13 02:38 | avalos | New Issue | |
| 2024-02-13 02:38 | avalos | Status | new => assigned |
| 2024-02-13 02:38 | avalos | Assigned To | => Florian Dold |
| 2024-02-13 17:27 | Florian Dold | Target Version | git (master) => 0.10 |
| 2024-03-05 23:43 | Christian Grothoff | Note Added: 0021707 | |
| 2024-03-05 23:43 | Christian Grothoff | Status | assigned => closed |
| 2024-03-05 23:43 | Christian Grothoff | Resolution | open => won't fix |
