View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0009683 | Taler | wallet (Android App) | public | 2025-04-01 11:08 | 2025-04-10 17:19 |
| Reporter | Antoine A | Assigned To | Florian Dold | ||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | assigned | Resolution | open | ||
| Target Version | 1.0 | ||||
| Summary | 0009683: Opening an already completed withdrawal and confirming it produce an error | ||||
| Description | { "code": 7001, "hint": "unexpected exception (message: unable to confirm withdrawal in current state: 83951617)", "message": null, "when": { "t_ms": 1743498486168 }, "stack": " at <anonymous> (<compiled_js>:35938)\n at <anonymous> (<compiled_js>:33859)\n" } | ||||
| Tags | No tags attached. | ||||
|
|
I've improved the error reporting, but IMO this is a UI problem. The UI should not let the user confirm, but instead check the transaction's state first. |
|
|
How do you confirm an already confirmed withdrawal? Not clear to me how to reproduce this. |
|
|
Initiate a withdrawal from the bank account and save the QR code somewhere. With your android wallet scan the QR code and complete the withdrawal. Then scan the QR code again, the wallet does not detect that the withdrawal is already confirmed and propose to confirm it again leading to the error. |
|
|
Shouldn't `getWithdrawalDetailsForUri` do the error reporting? |
|
|
What are the statuses where we can safely confirm the withdrawal? export type WithdrawalOperationStatusFlag = | "pending" | "selected" | "aborted" | "confirmed"; |
|
|
Not yet familiar enough with the codebase to fix this issue myself but I added some FIXME 0009683 in withdrawal tests where the current behavior should be improved |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2025-04-01 11:08 | Antoine A | New Issue | |
| 2025-04-01 11:08 | Antoine A | Status | new => assigned |
| 2025-04-01 11:08 | Antoine A | Assigned To | => Florian Dold |
| 2025-04-07 21:47 | Florian Dold | Note Added: 0024435 | |
| 2025-04-07 21:47 | Florian Dold | Assigned To | Florian Dold => avalos |
| 2025-04-09 22:07 | avalos | Note Added: 0024492 | |
| 2025-04-09 22:08 | avalos | Assigned To | avalos => Antoine A |
| 2025-04-09 22:08 | avalos | Status | assigned => feedback |
| 2025-04-10 09:01 | Antoine A | Note Added: 0024493 | |
| 2025-04-10 09:01 | Antoine A | Assigned To | Antoine A => avalos |
| 2025-04-10 13:15 | avalos | Note Added: 0024496 | |
| 2025-04-10 14:04 | avalos | Note Added: 0024498 | |
| 2025-04-10 14:04 | avalos | Assigned To | avalos => Florian Dold |
| 2025-04-10 17:19 | Antoine A | Note Added: 0024500 | |
| 2025-04-10 17:19 | Antoine A | Status | feedback => assigned |
