View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0007676Talerwallet (Android App)public2023-02-11 07:402023-02-23 17:05
Reporteravalos Assigned Togrote  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Platformarm64OSLineageOSOS Version18.1
Product Version0.9.1 
Target Version0.9.2Fixed in Version0.9.2 
Summary0007676: “Confirm with bank” UX should be clearer
DescriptionThis was discussed already before! When creating a withdrawal, the user is expected to confirm the withdrawal with the bank. Users might switch to the app before the browser shows the confirmation challenge, and the withdrawal transaction in the withdrawal list will only shows a “Confirm with bank” message, leaving the user wondering: “what should I do?” Unless they click the transaction and see the “Confirm with bank” button, they might not even know they need to do anything at all. The transaction might be left unconfirmed for eternity, and by then, I doubt humankind will continue existing.

(Note: ideally, the confirmation button should be shown for each transaction replacing in the same place where we put the KYC button.)
Steps To Reproduce1. Withdraw from the bank.
2. Switch to the app before the challenge shows up.
3. Observe the “Confirm with bank” message.
4. Panic (if you're a regular user) or leave it that way.
Tagsfeedback videos

Activities

sebasjm

2023-02-13 20:00

developer   ~0019827

What wallet web extension does is use the transactionId from `WalletApiOperation.AcceptManualWithdrawal` or `WalletApiOperation.AcceptBankIntegratedWithdrawal` to redirect the user to the transaction details and not the transaction list.

If there is a better approach it will be nice to be consistent, let me know! :)

avalos

2023-02-17 21:03

developer   ~0019873

That's also what the Android app does, I believe.

Christian Grothoff

2023-02-19 10:56

manager   ~0019880

Does now, or used to do? In the past, it did not...

avalos

2023-02-21 18:39

developer   ~0019897

Last edited: 2023-02-21 18:40

 This is how it looks like with the action button in dev/ivan-avalos/master branch.
Screenshot_20230221-113715_Taler_Wallet_Debug.png (89,407 bytes)   
Screenshot_20230221-113715_Taler_Wallet_Debug.png (89,407 bytes)   

Christian Grothoff

2023-02-21 18:45

manager   ~0019898

I see. Not bad. Still not sure if it is ideal. However, I do hope we still jump to the transaction detail page instead of the transaction list, right?

grote

2023-02-22 13:08

developer   ~0019901

resolved with https://git.taler.net/taler-android.git/commit/?id=4559c84e445ea43957d19b2022c856c9625a5fc1 and https://git.taler.net/taler-android.git/commit/?id=79d52ac4960e99d7b67f6435b03b010ed536e161

Issue History

Date Modified Username Field Change
2023-02-11 07:40 avalos New Issue
2023-02-11 07:40 avalos Status new => assigned
2023-02-11 07:40 avalos Assigned To => avalos
2023-02-11 07:40 avalos Tag Attached: feedback videos
2023-02-13 20:00 sebasjm Note Added: 0019827
2023-02-17 21:03 avalos Note Added: 0019873
2023-02-19 10:56 Christian Grothoff Note Added: 0019880
2023-02-21 18:39 avalos Note Added: 0019897
2023-02-21 18:39 avalos File Added: Screenshot_20230221-113715_Taler_Wallet_Debug.png
2023-02-21 18:40 avalos Note Edited: 0019897
2023-02-21 18:45 Christian Grothoff Note Added: 0019898
2023-02-22 13:08 grote Assigned To avalos => grote
2023-02-22 13:08 grote Status assigned => resolved
2023-02-22 13:08 grote Resolution open => fixed
2023-02-22 13:08 grote Note Added: 0019901
2023-02-23 17:05 Christian Grothoff Status resolved => closed
2023-02-23 17:05 Christian Grothoff Fixed in Version => 0.9.2