View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0006459||Taler||mechant backend||public||2020-08-03 14:17||2020-09-05 13:46|
|Reporter||Christian Grothoff||Assigned To||Florian Dold|
|Platform||i7||OS||Debian GNU/Linux||OS Version||squeeze|
|Product Version||git (master)|
|Summary||0006459: show full contract wallet-style in merchant HTML page|
Assigning to Florian so he can eval if existing Wallet code can easily plug this. If not, bump to 0.8.1 and we'll take care of it then via mustach.
Location in the code is marked with a FIXME and the bug number.
|Tags||No tags attached.|
Contracts are currently rendered via React, i.e. by client-side DOM manipulation. That's necessary in the WebExtension wallet, as in a web extension we don't have any server to render stuff for us ;-)
||I understand that this won't work with JS anymore, that's why I suggested this as a temporary / quick fix (which means additional dependencies can also be OK). I don't think we'll have the time to do a proper (!) and fully equivalent MUST-based rendering for 0.8, and I don't think we want to show the raw JSON or _nothing_ for 0.8. So temporarily dropping in a JS-based solution seemed like an acceptable trade-off.|
I don't think this is viable, because the wallet's rendering of the contract terms is very minimalist right now. The functionality we get from (temporarily!) integrating the JS code in the merchant backend HTML page isn't justified.
It would be more effective to pass the contract JSON into the mustache template. The mustac library can do this for another JSON library already (json-c): https://gitlab.com/jobol/mustach/-/blob/master/mustach-json-c.h
I'll work on making jansson work with mustach.
Putting this on feedback as the following questions are open:
1. Do we really need the label mechanism in the contract terms, or can we inline labels for better rendering, and introduce a tag for a "URL location"
2. The "Location" definition seems incomplete, shouldn't we at least have additional "free form" address lines?
3. What parts of the contract terms should be rendered on the page? Should we really list auditors/exchanges and h_wire? Is the following list enough?
* product list (with pictures if available)
* merchant information
should probably be enough?
I would allow seeing auditors/exchanges and locations via buttons to 'expand' certain details (similar to what we used to do in some wallets).
|2020-08-03 14:17||Christian Grothoff||New Issue|
|2020-08-03 14:17||Christian Grothoff||Status||new => assigned|
|2020-08-03 14:17||Christian Grothoff||Assigned To||=> Florian Dold|
|2020-08-03 14:21||Florian Dold||Note Added: 0016556|
|2020-08-03 14:21||Florian Dold||Assigned To||Florian Dold => Christian Grothoff|
|2020-08-03 14:24||Christian Grothoff||Assigned To||Christian Grothoff => Florian Dold|
|2020-08-03 14:26||Christian Grothoff||Note Added: 0016557|
|2020-08-20 16:37||Florian Dold||Note Added: 0016685|
|2020-08-21 17:57||Florian Dold||Status||assigned => feedback|
|2020-08-21 17:57||Florian Dold||Note Added: 0016686|
|2020-08-21 20:52||Christian Grothoff||Note Added: 0016694|
|2020-08-21 20:52||Christian Grothoff||Status||feedback => assigned|
|2020-08-21 20:52||Christian Grothoff||Note Edited: 0016694||View Revisions|
|2020-09-05 13:46||Christian Grothoff||Target Version||0.8 => 0.8.1|