View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0005191||Taler||py bank (demonstrator, obsolete)||public||2017-12-06 12:46||2018-04-15 20:34|
|Reporter||Christian Grothoff||Assigned To||Marcello Stanisci|
|Platform||i7||OS||Debian GNU/Linux||OS Version||squeeze|
|Product Version||git (master)|
|Target Version||0.5||Fixed in Version||0.5|
|Summary||0005191: Python bank needs to implement /reject API (and related updates)|
|Description||I just pushed 95077e8..a6202bb (to api.git). The change(s) make a few minor modifications to /admin/add/incoming (in particular, wtid is now 'subject' and can be an arbitrary string) and /history (direction is mandatory, cancel+/- become possible directions) and adds the new /reject API.|
These changes are currently being implemented on the exchange side to support the new wirewatch utility properly, and to cancel wire transfers (via /reject) with invalid wire transfer subjects. I will "soon" push a version of the exchange that includes testcases for this new API and that will *require* the bank to implement it so that the exchange continues to work!
Please make the necessary (likely minor) modifications to /admin/add/incoming and /history (and extend the database schema with a 'cancelled' flag for transactions), and add the new /reject handler.
|Tags||No tags attached.|
From this description, it is not clear how the sender gets their money back; is it the bank that wires money back to the sender as a /reject's side effect?
If yes, I guess both sender and receiver will see the second wire transfer in their history.
For example, after a cancellation is accomplished, does a sender see something like the following,
in their history?
j: 1KUDOS to Exchange (cancelled)
j+k: 1KUDOS from Exchange
Anyways, do we really need this, instead of the Exchange simply wiring back money when it does not like something?
In the visible history (not /history), the original transfer from Alice to the Exchange may show (possibly optional, but for the first iteration it's probably easier to always show it and not add a complicated option to hide it), but with a line through it (strike out) to show that it was voided. As the transaction is nullified, it means it should no longer have any impact on either balance.
for how to do this in HTML3/4/5 compatible ways (using both <strike> and CSS).
recap: main logic is done; /history response still needs to admit the new 'cancel+/-' flag, and CSS strike trick is also missing.
||Implementing CSS strikethrough in 77b7a1d. This commit concludes this issue. There is a "strike me" link beside any history entry in a user's profile page where one can test this feature; to be soon removed.|
|2017-12-06 12:46||Christian Grothoff||New Issue|
|2017-12-06 12:46||Christian Grothoff||Status||new => assigned|
|2017-12-06 12:46||Christian Grothoff||Assigned To||=> Marcello Stanisci|
|2017-12-06 12:47||Christian Grothoff||Relationship added||related to 0005172|
|2017-12-06 12:47||Christian Grothoff||Relationship added||related to 0005077|
|2017-12-07 17:46||Marcello Stanisci||Note Added: 0012634|
|2017-12-07 18:13||Christian Grothoff||Note Added: 0012635|
|2017-12-09 11:15||Christian Grothoff||Relationship added||related to 0005200|
|2017-12-11 15:02||Marcello Stanisci||Note Added: 0012664|
|2017-12-11 15:02||Marcello Stanisci||Note Edited: 0012664|
|2017-12-13 17:01||Marcello Stanisci||Status||assigned => resolved|
|2017-12-13 17:01||Marcello Stanisci||Resolution||open => fixed|
|2017-12-13 17:01||Marcello Stanisci||Note Added: 0012673|
|2017-12-13 17:13||Christian Grothoff||Fixed in Version||=> 0.5|
|2018-04-15 20:34||Christian Grothoff||Status||resolved => closed|
|2022-08-23 20:26||Christian Grothoff||Category||bank (demonstrator) => py bank (demonstrator, obsolete)|