Code review comment for lp:~camptocamp/sale-reports/sale-reports-add-so-webkit

Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote :

LGTM

I just think the dependancies on 'base' and 'report_webkit' modules aren't necessary in __openerp__.py as already inherited from base_header_webkit

review: Approve (code and spelling)

« Back to merge proposal