View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0010695 | Taler | merchant backoffice SPA | public | 2025-12-01 19:29 | 2025-12-04 22:41 |
| Reporter | htgoebel | Assigned To | sebasjm | ||
| Priority | high | Severity | major | Reproducibility | always |
| Status | assigned | Resolution | open | ||
| Product Version | 1.0 | ||||
| Target Version | 1.3 | ||||
| Summary | 0010695: Can create refund for wired order | ||||
| Description | Using the API one can create refunds for orders that have been wired already, see screenshot. Expected: To get Gone or another error message. | ||||
| Steps To Reproduce | - In the SPA create a payment with short deadlines and pay it. - Wait for the wire transfer deadline to be reached. - Use curl or another tools to create a refund for this order using the API. ==> This will return 200 and a MerchantRefundResponse - Go to the SPA, reload and view thise order's details ==> A "refund" will be shown | ||||
| Tags | No tags attached. | ||||
| Attached Files | |||||
|
|
The BIG issue here is that the merchant backend SPA fails to render the "refund_pending" field of the backend response: https://docs.taler.net/core/api-merchant.html#inspecting-orders As a result, the viewer cannot distinguish refunds that were *approved* from those that *succeeded*. We can argue that refunds after the wire deadline should not be approved anymore (easy to implement), but more importantly the SPA needs to distinguish clearly between refunds that did happen (wallet got the money) and those that were approved but the wallet did NOT get the money (for whatever reason). |
|
|
Merchant adjusted to 410 past wire deadline in b9c64ba7..4f8b3cf9 |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2025-12-01 19:29 | htgoebel | New Issue | |
| 2025-12-01 19:29 | htgoebel | File Added: Screenshot_20251201_192837.png | |
| 2025-12-02 11:14 | Stefan | Relationship added | related to 0010630 |
| 2025-12-02 23:00 | Christian Grothoff | Assigned To | => Christian Grothoff |
| 2025-12-02 23:00 | Christian Grothoff | Status | new => assigned |
| 2025-12-03 14:49 | vecirex | Relationship deleted | related to 0010630 |
| 2025-12-04 21:59 | Christian Grothoff | Priority | normal => high |
| 2025-12-04 21:59 | Christian Grothoff | Product Version | => 1.0 |
| 2025-12-04 21:59 | Christian Grothoff | Target Version | => 1.3 |
| 2025-12-04 22:16 | Christian Grothoff | Note Added: 0026743 | |
| 2025-12-04 22:17 | Christian Grothoff | Assigned To | Christian Grothoff => sebasjm |
| 2025-12-04 22:17 | Christian Grothoff | Severity | minor => major |
| 2025-12-04 22:17 | Christian Grothoff | Category | merchant backend => merchant backoffice SPA |
| 2025-12-04 22:30 | Christian Grothoff | Note Added: 0026744 | |
| 2025-12-04 22:37 | Christian Grothoff | Relationship added | related to 0010726 |
| 2025-12-04 22:40 | Christian Grothoff | Relationship added | related to 0010724 |
| 2025-12-04 22:41 | Christian Grothoff | Relationship added | related to 0010725 |