View Issue Details

IDProjectCategoryView StatusLast Update
0007449Talerwallet-corepublic2022-11-11 14:42
Reportersebasjm Assigned ToFlorian Dold 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
PrioritynormalSeveritytextReproducibilityhave not tried
Status closedResolutionwon't fix 
Product Versiongit (master) 
Target Version0.9.1Fixed in Version0.9.1 
Summary0007449: rename taler actions
DescriptionRenames are:

pay-push => transfer
pay-pull => invoice
exchange => trust-exchange
auditor => trust-auditor
recovery => restore

This should be:
1.- agree that this renames are fine
2.- add support for new names in wallet and merchant, both styles supported, deprecating old
3.- remove support for old name after some major release
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2022-11-11 10:08

manager   ~0019401

Last edited: 2022-11-11 10:12

transfer is already used for wire-transfers, might be confused with those. 'offer' might be an alternative (but we use that with tips (offering a tip) already). We could also just keep 'push' and 'pull' (without -pay) to keep it symmetric.

Overall, not yet convinced that changing the action names is at all justified: minimal gain if any, more work.

Christian Grothoff

2022-11-11 14:41

manager   ~0019403

Florian writes: Agreed, I don't like any of the other proposed names, and we shouldn't bike-shed here. The "pay-pull" and "pay-push" terminology also fits nicely into the "URIs as pointers to resources" concept, since the URI really identifies a p2p pull/push payment that the receiving party interacts with. -F

So let's resolve this one as "wont fix".

Issue History

Date ModifiedUsernameFieldChange
2022-11-10 19:56sebasjmNew Issue
2022-11-10 19:56sebasjmStatusnew => assigned
2022-11-10 19:56sebasjmAssigned To => Florian Dold
2022-11-11 10:08Christian GrothoffNote Added: 0019401
2022-11-11 10:09Christian GrothoffNote Edited: 0019401
2022-11-11 10:12Christian GrothoffNote Edited: 0019401
2022-11-11 14:41Christian GrothoffNote Added: 0019403
2022-11-11 14:42Christian GrothoffSeverityminor => text
2022-11-11 14:42Christian GrothoffStatusassigned => closed
2022-11-11 14:42Christian GrothoffResolutionopen => won't fix
2022-11-11 14:42Christian GrothoffProduct Version => git (master)
2022-11-11 14:42Christian GrothoffTarget Version => 0.9.1
2022-11-11 14:42Christian GrothoffFixed in Version => 0.9.1
2023-04-13 18:36Florian DoldCategorywallet (TS core) => wallet-core