View Issue Details

IDProjectCategoryView StatusLast Update
0009367Talerwallet-corepublic2025-01-10 12:18
ReporterFlorian Dold Assigned ToFlorian Dold  
PrioritynormalSeverityminorReproducibilityhave not tried
Status resolvedResolutionfixed 
Target Version1.0 
Summary0009367: add flag to balance-change notification for transaction that change the user-visible balance
DescriptionWe use the notifications for various purposes, telling the UI that the user-visible balance changed is just one of them.

To prevent unnecessary UI updates, the notification should have an extra flag to indicate if a user-visible balance was affected.

The notification will still be emitted when the material balance changes (not directly visible to the user!), but the flag won't be set.
TagsNo tags attached.

Activities

Florian Dold

2025-01-07 16:07

manager   ~0023942

We still emit non-internal balance change notifications for refresh:

{"type":"notification","payload":{"type":"balance-change","hintTransactionId":"txn:refresh:TMK2AN1MZQF5TCBZG0R7NJ50DYY9XDDKB7TVRX8DV2FJQM6M1M70","isInternal":false}}

We should only do that if refresh fails.

Florian Dold

2025-01-10 12:18

manager   ~0023966

commit ed409e0d56e913c8a999aa5e92afc4e6b15c570e
Author: Florian Dold <florian@dold.me>
Date: Thu Jan 9 17:52:16 2025 +0100

    wallet-core: add better tracking of balance change effects
    
    We now keep track whether a transaction transition (a) doesn't change
    the balance, (b) changes the balance, but preserves the user-visible
    balance or (c) can also change the user-visible balance.

Issue History

Date Modified Username Field Change
2024-12-04 12:02 Florian Dold New Issue
2024-12-04 12:02 Florian Dold Status new => assigned
2024-12-04 12:02 Florian Dold Assigned To => Florian Dold
2025-01-07 16:07 Florian Dold Note Added: 0023942
2025-01-10 12:18 Florian Dold Status assigned => resolved
2025-01-10 12:18 Florian Dold Resolution open => fixed
2025-01-10 12:18 Florian Dold Note Added: 0023966