|
|
@sebasjm I see that there is purse_expiration now added in https://git.taler.net/wallet-core.git/commit/?id=5c742afb However, the API docs still don't contain this. I can try to scrape the source code for details, but `any` in the docs should really go away in favor of real docs. |
|
|
> it must be possible for the user to specify an expiration time for the push payment, as only then the coins that were pushed can be recovered if the receiver didn't accept the payment. @Christian Grothoff it looks like before wallet-core was setting this on its on to a sane default value. I am not sure cluttering the UI with this is actually a good idea. Is there a strong use-case for customizing the expiry or could we not just add a small print saying: The money will be available for X days and if not claimed will return automatically (minus a nominal fee). |
|
|
@sebasjm it also seems in this commit, the amount was moved into the contract terms which is also not reflected in the documentation: https://docs.taler.net/wallet/wallet-core.html#initiatepeerpushpaymentop Also, this breaks peer-push in the Android wallet, because it was yet another unannounced protocol change. |
|
|
docs updated, should be online soon |
|
|
Christian, if you feel strongly about letting the user specify expiry, do you have special requirements for how to let the user do that? several options as buttons with one pre-selected or a date picker? or an input field for a number of days? |
|
|
No super-strong feelings. But: 1) I suspect we don't need time to be very precise in the default implementation, so *date* should be enough, not the exact hour / minute / second. 2) We probably don't want to offer excessively long choices --- like in 10 years makes no sense. In fact, probably even 1 year will not make sense. I'd expect all sane choices to be < 3 months (as we're talking about locking up funds!). So based on this, an input field with a number of days limited to 2 or maybe 3 digits is probably the simplest widget and one that will prevent stupid inputs, too ;-). |
|
|
This is my proposal. Expiration Date.png (14,190 bytes) |
|
|
Looks good. Fine with me. Maybe put the entire widget into a re-usable composable. |
|
|
implemented in https://git.taler.net/taler-android.git/commit/?id=1c979ef1d0efd8bdaed7dda292825c41f1d48893 |
- Anonymous
