Code review comment for lp:~camptocamp/account-financial-tools/7.0-add_l10n_fr_siret

Revision history for this message
Numérigraphe (numerigraphe) wrote :

I suggest you add a dependency on 'base' to make sure you can access view etc, as stated in http://doc.openerp.com/v6.1/developer/03_modules_1.html#openerp-module-descriptor-file-openerp-py.
Other than that I think it's OK.
Lionel

review: Needs Fixing

« Back to merge proposal