View Issue Details

IDProjectCategoryView StatusLast Update
0007449Talerwallet-corepublic2022-11-11 15:42
Reportersebasjm Assigned ToFlorian Dold  
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.

Activities

Christian Grothoff

2022-11-11 11:08

manager   ~0019401

Last edited: 2022-11-11 11: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 15: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 Modified Username Field Change
2022-11-10 20:56 sebasjm New Issue
2022-11-10 20:56 sebasjm Status new => assigned
2022-11-10 20:56 sebasjm Assigned To => Florian Dold
2022-11-11 11:08 Christian Grothoff Note Added: 0019401
2022-11-11 11:09 Christian Grothoff Note Edited: 0019401
2022-11-11 11:12 Christian Grothoff Note Edited: 0019401
2022-11-11 15:41 Christian Grothoff Note Added: 0019403
2022-11-11 15:42 Christian Grothoff Severity minor => text
2022-11-11 15:42 Christian Grothoff Status assigned => closed
2022-11-11 15:42 Christian Grothoff Resolution open => won't fix
2022-11-11 15:42 Christian Grothoff Product Version => git (master)
2022-11-11 15:42 Christian Grothoff Target Version => 0.9.1
2022-11-11 15:42 Christian Grothoff Fixed in Version => 0.9.1
2023-04-13 20:36 Florian Dold Category wallet (TS core) => wallet-core