View Issue Details

IDProjectCategoryView StatusLast Update
0007436Talerwallet (WebExtension)public2023-01-26 21:53
ReporterFlorian Dold 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
PrioritynormalSeveritytweakReproducibilityhave not tried
Status closedResolutionfixed 
Product Version0.9 
Target Version0.9.1Fixed in Version0.9.1 
Summary0007436: manual withdrawal with IBAN is rendered weirdly
DescriptionFirst of all, I didn't find where to copy the payto:// URI for this directly.

Then, it shows "SANDBOXX/DE159593" as the IBAN, which is not a valid IBAN.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2022-11-06 22:13

manager   ~0019359

payto://-URI parser should split of the BIC from the IBAN, and UIs should allow both to be copied separately.

Christian Grothoff

2022-11-06 22:13

manager   ~0019360

We should check that Android is doing this correctly as well, before resolving the bug.

sebasjm

2022-11-07 13:39

developer   ~0019372

BIC and IBAN is rendered in different fields since commit 661469f8

should we show payto uri? doesn't seem like a normal user will prefer that over the account information.
maybe the display it under devmode?

Christian Grothoff

2022-11-07 14:26

manager   ~0019374

We should also show the payto://-URI (and/or allow copying it to the clipboard), as our bank UI does support sending money to a payto://-URI explicitly already. And we want to promote RFC 8905 ;-).

But, it doesn't need to be too prominent, so I'd show it below the other information in a separate section or box. (I'd show it with a link to RFC 8905 in the label, a copy-to-clipboard button and the payto://-URI as text).

sebasjm

2022-11-07 18:12

developer   ~0019383

roger that

fixed at ca2b4d8c80d208038e99d3b7be8d8e398dc71067

Issue History

Date ModifiedUsernameFieldChange
2022-11-04 19:51Florian DoldNew Issue
2022-11-04 19:51Florian DoldStatusnew => assigned
2022-11-04 19:51Florian DoldAssigned To => Florian Dold
2022-11-06 22:12Christian GrothoffAssigned ToFlorian Dold => sebasjm
2022-11-06 22:13Christian GrothoffNote Added: 0019359
2022-11-06 22:13Christian GrothoffSeverityminor => tweak
2022-11-06 22:13Christian GrothoffProduct Version => 0.9
2022-11-06 22:13Christian GrothoffTarget Version => 0.9.1
2022-11-06 22:13Christian GrothoffNote Added: 0019360
2022-11-07 13:39sebasjmNote Added: 0019372
2022-11-07 13:39sebasjmStatusassigned => feedback
2022-11-07 13:39sebasjmAssigned Tosebasjm => Florian Dold
2022-11-07 14:26Christian GrothoffNote Added: 0019374
2022-11-07 14:26Christian GrothoffAssigned ToFlorian Dold => sebasjm
2022-11-07 18:12sebasjmStatusfeedback => informing
2022-11-07 18:12sebasjmResolutionopen => fixed
2022-11-07 18:12sebasjmNote Added: 0019383
2022-11-18 21:42Christian GrothoffFixed in Version => 0.9.1
2023-01-26 21:53Christian GrothoffStatusinforming => closed
2023-04-13 18:37Florian DoldCategorywallet (WebExtensions) => wallet (WebExtension)