View Issue Details

IDProjectCategoryView StatusLast Update
0009767Talerexchangepublic2025-04-22 22:01
ReporterChristian Grothoff Assigned Tosebasjm  
PriorityhighSeverityfeatureReproducibilityN/A
Status resolvedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version1.0 
Summary0009767: filter rules by payto wire method
DescriptionBALANCE, MERGE/P2P_RECEIVE only apply to Taler-wallet payto://-URIs,
DEPOSIT, AGGREGATE, WITHDRAW, RESERVE_CLOSE only apply to bank accounts
TRANSACTION, REFUND applies to both.

We should store, return (to KYC SPA) and edit (AML SPA) only the rules that are applicable to a payto:// wire method!
TagsNo tags attached.

Activities

Christian Grothoff

2025-04-21 20:06

manager   ~0024639

The attached screenshot shows what happens otherwise: here we need the user to fill in the "Form" (bank account), but we show rules that would only apply to wallets, possibly causing the user to try to satisfy the wrong requirement!
bad-rules.png (49,921 bytes)   
bad-rules.png (49,921 bytes)   

Christian Grothoff

2025-04-21 21:15

manager   ~0024640

taler-docs.git::bb475832..352c11eb adds an "is_wallet" to the AML /decisions response, so that the AML-SPA can tell which rules should be given as choices.

Christian Grothoff

2025-04-21 21:17

manager   ~0024641

exchange.git :: c64322d5f..a1e12a191 implements the 'is_wallet' extension of /decisions.
Still does NOT actually filter is own rules (TBD!).

Christian Grothoff

2025-04-22 13:00

manager   ~0024647

3665a932b..19173147e completes the implementation in the exchange backend. SPA update (& QC) missing on this.

sebasjm

2025-04-22 22:00

developer   ~0024650

commit a458ec3b5558908d3176cce4178b6c6bf7926938 (HEAD -> master, origin/master, origin/HEAD)
Author: Sebastian <sebasjm@gmail.com>
Date: Tue Apr 22 16:59:06 2025 -0300

    add warning based on account type, also hide buttons if not needed and hint in the title the type of account

sebasjm

2025-04-22 22:01

developer   ~0024651

i think that aml spa was the missing piece

Issue History

Date Modified Username Field Change
2025-04-21 20:05 Christian Grothoff New Issue
2025-04-21 20:05 Christian Grothoff Status new => assigned
2025-04-21 20:05 Christian Grothoff Assigned To => Christian Grothoff
2025-04-21 20:06 Christian Grothoff Note Added: 0024639
2025-04-21 20:06 Christian Grothoff File Added: bad-rules.png
2025-04-21 21:15 Christian Grothoff Note Added: 0024640
2025-04-21 21:17 Christian Grothoff Note Added: 0024641
2025-04-22 12:44 Christian Grothoff Description Updated
2025-04-22 13:00 Christian Grothoff Note Added: 0024647
2025-04-22 13:00 Christian Grothoff Assigned To Christian Grothoff => sebasjm
2025-04-22 22:00 sebasjm Note Added: 0024650
2025-04-22 22:01 sebasjm Status assigned => resolved
2025-04-22 22:01 sebasjm Resolution open => fixed
2025-04-22 22:01 sebasjm Note Added: 0024651