View Issue Details

IDProjectCategoryView StatusLast Update
0007768Talerlibeufin-bank webuipublic2023-09-23 13:09
ReporterMS 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
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Target Version0.9.3Fixed in Version0.9.3 
Summary0007768: cash-out rates wrong when starting from fiat.
DescriptionWhen the cash-out amount is calculated starting from the fiat, then the UI
submits a rates/fees application that's not accepted by the backend.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

sebasjm

2023-03-15 12:46

developer   ~0019964

the truncation of amount after applying rate conversion is wrong, here [1] SPA is doing a simple string truncation and libeufin does a BigDecimal round up, SPA should match that [2]

if calculateFromCredit with amount 3.15 sellFee 0 sellRate 0.95 I get amount debit debitAmount 3.31 which is wrong because calculateFromCredit for amount 3.31 (same rates) the credit is 3.14 !== 3.15

Amounts API in taler-util doesnt have a truncation or roundup function, should we add this function in amounts API?


[1] https://git.taler.net/wallet-core.git/tree/packages/demobank-ui/src/pages/BusinessAccount.tsx#n839
[2] https://git.taler.net/libeufin.git/tree/sandbox/src/main/kotlin/tech/libeufin/sandbox/CircuitApi.kt#n404

sebasjm

2023-03-31 22:16

developer   ~0019998

using the cashouts/estimate endpoint

sebasjm

2023-03-31 22:16

developer   ~0019999

fixed at f947c8e54919343ac4f5f951d2f701651c06dd52

Issue History

Date ModifiedUsernameFieldChange
2023-03-14 11:43MSNew Issue
2023-03-14 11:43MSStatusnew => assigned
2023-03-14 11:43MSAssigned To => sebasjm
2023-03-15 12:46sebasjmNote Added: 0019964
2023-03-31 22:16sebasjmNote Added: 0019998
2023-03-31 22:16sebasjmStatusassigned => informing
2023-03-31 22:16sebasjmResolutionopen => fixed
2023-03-31 22:16sebasjmNote Added: 0019999
2023-05-04 18:06Christian GrothoffFixed in Version => 0.9.3
2023-05-04 18:06Christian GrothoffTarget Version => 0.9.3
2023-09-23 13:09Christian GrothoffStatusinforming => closed
2023-12-03 00:21Christian GrothoffCategorydemobank-ui => libeufin-bank-ui
2024-01-12 13:10Christian GrothoffCategorylibeufin-bank-ui => libeufin-bank-ui (SPA)
2026-08-23 14:18Florian DoldCategorylibeufin-bank-ui (SPA) => libeufin-bank webui