View Issue Details

IDProjectCategoryView StatusLast Update
0010042Talerwallet (Android App)public2025-08-31 17:09
Reportersebasjm Assigned Toavalos 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
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version1.0 
Target Version1.0 stretch goalsFixed in Version1.0 stretch goals 
Summary0010042: failed to create an invoice
Descriptioni've got unable to initiate pull payment from ...

maybe because for data is lost after accepting tos
Steps To Reproducefrom empty wallet
withdraw demo money
create an invoice (p2p receive)
set amount and purpose
click accpt tops
click accept tops
click create request
Tagsux

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2025-05-28 18:22

manager   ~0025042

I'm confused, the sequence seems already wrong. Why are we allowed to create an invoice BEFORE accepting ToS? It should be:

0) from empty wallet
[ withdraw demo money ] -- irrelevant here, but of course need to add the exchange somehow without ToS acceptance, so this is *a* way.
1) click "create an invoice" (p2p receive)
2) show ToS and ask user to accept tops
3) set amount and purpose ( + expiration) + click create request

Everything else is just wrong! Please fix in all wallets!

MarcS

2025-05-28 18:51

developer   ~0025045

On iOS, of course you first get asked to accept the ToS before you type the amount...

avalos

2025-06-03 14:03

developer   ~0025105

Fixed in master (Android), will be released soon.

commit 6d3efa869b6889d3b5d825eb998a7c8beb3c397c
Author: Iván Ávalos <avalos@disroot.org>
Date: Thu May 29 17:21:18 2025 +0200

    [wallet] important ToS review fixes
image.png (38,305 bytes)   
image.png (38,305 bytes)   

Issue History

Date ModifiedUsernameFieldChange
2025-05-28 10:26sebasjmNew Issue
2025-05-28 18:22Christian GrothoffNote Added: 0025042
2025-05-28 18:22Christian GrothoffStatusnew => implementing
2025-05-28 18:22Christian GrothoffProduct Version => 1.0
2025-05-28 18:22Christian GrothoffTarget Version => 1.0 stretch goals
2025-05-28 18:22Christian GrothoffTag Attached: ux
2025-05-28 18:23Christian GrothoffAssigned To => avalos
2025-05-28 18:23Christian GrothoffStatusimplementing => assigned
2025-05-28 18:51MarcSNote Added: 0025045
2025-06-03 14:03avalosNote Added: 0025105
2025-06-03 14:03avalosFile Added: image.png
2025-06-03 14:07avalosStatusassigned => informing
2025-06-03 14:07avalosResolutionopen => fixed
2025-06-03 17:52Christian GrothoffFixed in Version => 1.0 stretch goals
2025-08-31 17:09Christian GrothoffStatusinforming => closed