View Issue Details

IDProjectCategoryView StatusLast Update
0008716Talermerchant portal webuipublic2024-04-15 19:32
Reportersebasjm Assigned Tosebasjm Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
PrioritynormalSeveritytweakReproducibilitysometimes
Status closedResolutionfixed 
Product Versiongit (master) 
Target Version0.10Fixed in Version0.10 
Summary0008716: wrong refund deadline
Descriptiondoing some testing i found one order which from which merchant backend returned with contract_terms.refund_deadline.t_s = 0 the merchant SPA rendered as "refundable at 1969". SPA expect refund deadline after order creatiion or "never" to hide it.

the order was still payable (before pay deadline) but it was claim long after it was created.

current config:
instance default payment delay: 2h
instance default wire transfer delay: 2d
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2024-04-09 15:10

manager   ~0022169

I don't think this is wrong at all. The way I read it, the spec says that the refund_deadline field is *mandatory*. So I have to set it.
We also say that if the refund deadline is in the past, the order is simply *not refundable*. So 0 < any pay deadline, so not refundable.
I think you should just check if the refund deadline is in the past (and 1970 is definitively in the past!), and then not show 'refundable' at all, because it clearly was never refundable.

sebasjm

2024-04-09 16:22

developer   ~0022174

ACK updating in merchant SPA

sebasjm

2024-04-09 16:23

developer   ~0022175

7f3253333

Issue History

Date ModifiedUsernameFieldChange
2024-04-09 14:54sebasjmNew Issue
2024-04-09 14:54sebasjmStatusnew => assigned
2024-04-09 14:54sebasjmAssigned To => Christian Grothoff
2024-04-09 15:10Christian GrothoffNote Added: 0022169
2024-04-09 15:10Christian GrothoffAssigned ToChristian Grothoff => sebasjm
2024-04-09 15:10Christian GrothoffSeverityminor => tweak
2024-04-09 16:22sebasjmCategorymechant backend => merchant backoffice SPA
2024-04-09 16:22sebasjmNote Added: 0022174
2024-04-09 16:23sebasjmStatusassigned => informing
2024-04-09 16:23sebasjmResolutionopen => fixed
2024-04-09 16:23sebasjmNote Added: 0022175
2024-04-09 16:23sebasjmStatusinforming => feedback
2024-04-09 16:23sebasjmResolutionfixed => reopened
2024-04-09 16:23sebasjmTarget Version0.11 => 0.10
2024-04-09 16:23sebasjmStatusfeedback => informing
2024-04-09 16:23sebasjmResolutionreopened => fixed
2024-04-11 23:50Christian GrothoffFixed in Version => 0.10
2024-04-15 19:32Christian GrothoffStatusinforming => closed
2026-08-23 14:18Florian DoldCategorymerchant backoffice SPA => merchant portal webui