Code review comment for lp:~camptocamp/sale-reports/vre-sale_order_proformat

Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Hi, Vincent,

Thanks you very much for this module. It's something that I have also on my pendings. I have tested, and these are the issues I have found:

- Module doesn't install because you have on "from . import report", but there is no on that folder. Please remove this line.
- Module title put "ProFormat" instead of "Pro Forma"
- When I try to print the report, I get:

SyntaxError: JSON.parse: unexpected end of data at line 2 column 1 of the JSON data


review: Needs Fixing (code review and test)

« Back to merge proposal