| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0010695 | Taler | merchant backoffice SPA | public | 2025-12-01 19:29 | 2025-12-05 20:53 |
| Reporter | htgoebel | Assigned To | vlada.svirsh | ||
| Priority | high | Severity | major | Reproducibility | always |
| Status | assigned | Resolution | open | ||
| Product Version | 1.0 | ||||
| Target Version | 1.5 | ||||
| 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. | ||||
|
|
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 |
|
|
refund that did happen are shown as "refund taken" and refund that was approved by the merchant is shown as "refunded amount" also pending refund "pending" vs "taken" are shown green vs red In this screenshot you can see both with the fix of 0010725 where the refund URI is not there anymore after wire deadline happened. If you think that still confusing i can add a banner that says "the rest of the refund $ (refunded - taken)" image.png (27,961 bytes) image-2.png (94,967 bytes) |
|
|
i think the spa is clear |
|
|
Let's see if vlada comes up with an idea for how to make it clearer! |
