View Issue Details

IDProjectCategoryView StatusLast Update
0006551Talerwallet-corepublic2021-08-24 14:23
Reportergrote Assigned ToFlorian Dold Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Versiongit (master) 
Target Version0.8Fixed in Version0.8 
Summary0006551: Refund transactions show wrong amounts and reduce balance
DescriptionWhen doing three incremental refunds of a 10 TESTKUDOS purchase, all refund transactions always show the updated amount which is strange, because in the end it looks like you got +10 +10 +10, so 30 TESTKUDOS to your balance. Ideally, the amounts are shown in a way that they add up together to the total refunded amount.

Also while receiving the refund, the balance is dropping close to zero which looks very scary like you can see your wallet being drained. We should probably hide this refresh related balance change from the user.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Florian Dold

2020-09-01 14:41

manager   ~0016809

The wallet currently shows only one refund transaction, even if multiple refunds happened. This is certainly a bug in the wallet.

But it also looks like the UI is displaying transactions even though the wallet-core doesn't return them anymoe?

Florian Dold

2020-09-01 14:44

manager   ~0016810

Ah, no, this might actually only be a bug in wallet-core.

I'm updating the integration test ...

Florian Dold

2020-09-01 18:04

manager   ~0016811

These were two separate issues:

* Refund raw/effective amounts swapped. This one is fixed in v0.7.1-dev.24, and the test-refund-incremental integration test has been extended to check the transactions list.
* Amounts "locked up in refresh" were not shown as part of the available balance in parts of the withdrawal process. This *should* be mostly fixed in v0.7.1-dev.24 as well, but 0006554 tracks the status of more test cases for balance changes.

Issue History

Date ModifiedUsernameFieldChange
2020-09-01 13:41groteNew Issue
2020-09-01 13:41groteStatusnew => assigned
2020-09-01 13:41groteAssigned To => Florian Dold
2020-09-01 14:41Florian DoldNote Added: 0016809
2020-09-01 14:44Florian DoldNote Added: 0016810
2020-09-01 18:04Florian DoldNote Added: 0016811
2020-09-01 18:04Florian DoldStatusassigned => informing
2020-09-01 18:04Florian DoldResolutionopen => fixed
2020-09-02 07:52Christian GrothoffProduct Version => git (master)
2020-09-02 07:52Christian GrothoffFixed in Version => 0.8
2020-09-02 07:52Christian GrothoffTarget Version => 0.8
2021-08-24 14:23Christian GrothoffStatusinforming => closed
2023-04-13 18:36Florian DoldCategorywallet (TS core) => wallet-core