View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006591 | Taler | wallet (Android App) | public | 2020-09-08 21:10 | 2023-09-23 13:09 |
| Reporter | Florian Dold | Assigned To | avalos | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Target Version | 0.9.3 | Fixed in Version | 0.9.3 | ||
| Summary | 0006591: Android wallet should support aborting partially failed payments | ||||
| Description | The wallet-core API now supports aborting failed payments (and obtaining a refund of partial spends) via "abortFailedPayWithRefund". See 0006590 for general UX issues regarding this, especially for what happens if this refund fails (currently if will be retried "infinitely" with increasing retry timeout). | ||||
| Tags | No tags attached. | ||||
| Based on Mumble discussions: We should avoid making this too complex in the UI. => A pending payment should have one "cancel" button, and the "abortFailedPayWithRefund" should be replaced with a simpler "cancelPayment" method that "does the right thing" and recovers as much funds as possible. | |
| We now have a generic "abortTransaction" request and DD37 specifies the transaction lifecycle. | |
| What's the first version of wallet-core to support it? | |
| 0.9.1 definitely supports it | |
| This pretty much depends on DD37, which I'm already working on implementing in `dev/ivan-avalos/dd37` branch. I still need some wallet-core features to be implemented before I can finish my client-side DD37 work, but once it's all done, it will be possible to abort partially failed payments, only when DD37 says it's possible. | |
| This should be fixed now that the DD37 branch was merged. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2020-09-08 21:10 | Florian Dold | New Issue | |
| 2020-09-08 21:10 | Florian Dold | Status | new => assigned |
| 2020-09-08 21:10 | Florian Dold | Assigned To | => grote |
| 2020-10-08 15:08 | Florian Dold | Note Added: 0017004 | |
| 2020-10-08 15:08 | Florian Dold | Assigned To | grote => Florian Dold |
| 2023-02-17 01:35 | Florian Dold | Assigned To | Florian Dold => avalos |
| 2023-02-17 01:35 | Florian Dold | Note Added: 0019858 | |
| 2023-02-17 20:01 | avalos | Note Added: 0019872 | |
| 2023-02-19 22:32 | Florian Dold | Note Added: 0019883 | |
| 2023-02-19 22:32 | Florian Dold | Target Version | => 0.9.3 |
| 2023-04-11 16:26 | avalos | Note Added: 0020073 | |
| 2023-07-24 22:33 | avalos | Note Added: 0020381 | |
| 2023-07-24 22:33 | avalos | Status | assigned => informing |
| 2023-07-24 22:33 | avalos | Resolution | open => fixed |
| 2023-09-23 13:07 | Christian Grothoff | Fixed in Version | => 0.9.3 |
| 2023-09-23 13:09 | Christian Grothoff | Status | informing => closed |