View Issue Details

IDProjectCategoryView StatusLast Update
0007367Talerwallet (Android App)public2023-09-23 13:09
ReporterFlorian Dold 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 
Target Version0.9.3Fixed in Version0.9.3 
Summary0007367: Android wallet shows "confirm with bank" button even for confirmed withdrawals
DescriptionEven if a withdrawal has been confirmed with the bank (i.e. the 2FA step succeeded), the wallet shows the "confirm with bank" button.

I've confirmed in the adb logs that the transactions list does have "confirmed: true" in the response.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

grote

2022-09-23 16:20

reporter   ~0019174

Can this field now be relied upon in all circumstances? For some reason, the code still has this construct:

!pending && (
                (withdrawalDetails is TalerBankIntegrationApi && withdrawalDetails.confirmed) ||
                        withdrawalDetails is ManualTransfer
                )

Florian Dold

2023-04-04 16:12

manager   ~0020015

This seems to be fixed and working now.

Issue History

Date ModifiedUsernameFieldChange
2022-09-23 12:44Florian DoldNew Issue
2022-09-23 12:44Florian DoldStatusnew => assigned
2022-09-23 12:44Florian DoldAssigned To => grote
2022-09-23 16:20groteNote Added: 0019174
2022-10-20 09:08Christian GrothoffAssigned Togrote => Florian Dold
2022-10-20 09:08Christian GrothoffStatusassigned => feedback
2023-04-04 14:10Florian DoldTarget Version => 0.9.3
2023-04-04 16:12Florian DoldStatusfeedback => informing
2023-04-04 16:12Florian DoldResolutionopen => fixed
2023-04-04 16:12Florian DoldNote Added: 0020015
2023-09-23 13:07Christian GrothoffFixed in Version => 0.9.3
2023-09-23 13:09Christian GrothoffStatusinforming => closed