View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005283 | Taler | merchant backend | public | 2018-02-21 12:17 | 2019-12-20 18:12 |
| Reporter | Marcello Stanisci | Assigned To | Marcello Stanisci | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | git (master) | ||||
| Target Version | 0.6 | Fixed in Version | 0.6 | ||
| Summary | 0005283: New test cases lack code comments. | ||||
| Description | A big part of the "new styled" test instructions code needs comments. | ||||
| Tags | No tags attached. | ||||
| 1f4f0f5 comments merchant tip CMDs. | |
| merchant testing lib header file and "traits" C files TBD. | |
| This commits ends the merchant side: eb8549d94be9c36. Libraries and header files are now commented; the test cases themselves may lack verbose comments in a few places, but this is not so critical to track here. Leaving open to track the exchange side of it. | |
| 7a1146f147da544 @exchange has comments for CMDs and traits from the "lib" side. The _bank_-lib is still lacking comments. Once that part has been done, then the test suite themselves would benefit from more commenting, although that is not so critical to be kept tracked here. | |
| Several trait offering functions - at the exchange AND the merchant - have the old 'selector' argument in the @param line; to be changed to the new 'index' paramenter. | |
| Ah, the exchange will also have to get the header files adapted to the new comments. | |
| 3cb0454d comments exchange header files. | |
| 01f933bb @ exchange and 3304bdb @ merchant address the "@param selector" problem, and close this issue. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2018-02-21 12:17 | Marcello Stanisci | New Issue | |
| 2018-02-21 12:17 | Marcello Stanisci | Status | new => assigned |
| 2018-02-21 12:17 | Marcello Stanisci | Assigned To | => Marcello Stanisci |
| 2018-05-23 09:33 | Marcello Stanisci | Target Version | => 0.6 |
| 2018-05-23 09:34 | Marcello Stanisci | Note Added: 0012954 | |
| 2018-05-24 17:53 | Marcello Stanisci | Note Added: 0012958 | |
| 2018-05-25 10:07 | Marcello Stanisci | Note Added: 0012960 | |
| 2018-05-25 10:08 | Marcello Stanisci | Note Edited: 0012960 | |
| 2018-05-28 15:26 | Marcello Stanisci | Note Added: 0012964 | |
| 2018-05-28 20:17 | Marcello Stanisci | Note Added: 0012965 | |
| 2018-05-28 20:18 | Marcello Stanisci | Note Edited: 0012965 | |
| 2018-05-28 20:18 | Marcello Stanisci | Note Added: 0012966 | |
| 2018-05-29 10:19 | Marcello Stanisci | Note Added: 0012967 | |
| 2018-05-29 10:45 | Marcello Stanisci | Note Added: 0012968 | |
| 2018-05-29 10:46 | Marcello Stanisci | Status | assigned => informing |
| 2018-05-29 10:46 | Marcello Stanisci | Resolution | open => fixed |
| 2018-06-12 06:59 | Christian Grothoff | Product Version | => git (master) |
| 2018-06-12 06:59 | Christian Grothoff | Fixed in Version | => 0.6 |
| 2019-12-20 18:12 | Christian Grothoff | Status | informing => closed |
| 2025-03-22 13:09 | Christian Grothoff | Category | mechant backend => merchant backend |