View Issue Details

IDProjectCategoryView StatusLast Update
0010695Talermerchant backoffice SPApublic2025-12-04 22:41
Reporterhtgoebel Assigned Tosebasjm  
PriorityhighSeveritymajorReproducibilityalways
Status assignedResolutionopen 
Product Version1.0 
Target Version1.3 
Summary0010695: Can create refund for wired order
DescriptionUsing 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
TagsNo tags attached.
Attached Files
Screenshot_20251201_192837.png (19,471 bytes)   
Screenshot_20251201_192837.png (19,471 bytes)   

Relationships

related to 0010726 assignedFlorian Dold Wallet pseudo-accepts refunds even after wire deadline 
related to 0010724 assignedsebasjm Refund "pending" even after wire deadline 
related to 0010725 assignedsebasjm UX: Refund URL shown even after wire deadline 

Activities

Christian Grothoff

2025-12-04 22:16

manager   ~0026743

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).

Christian Grothoff

2025-12-04 22:30

manager   ~0026744

Merchant adjusted to 410 past wire deadline in b9c64ba7..4f8b3cf9

Issue History

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