Code review comment for lp:~numerigraphe/openobject-addons/trunk-l10n-eu-nace

Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

Hi Lionel,

The 2 nomenclature modules (l10n_eu_nace, l10n_fr_naf_ape) your are proposing would be a great addition to OpenERP Apps, but we'd rather not include them in the standard distribution (not to mention that l10n_eu_nace weighs 24Mb).

BTW, I think it's great that you included the source of the RAMON nomenclature with a conversion script, rather than simply the resulting XML output.

Thanks for your understanding and your excellent work in the community, it is greatly appreciated!

PS: for reference, our merge proposal acceptance policy is available at http://bit.ly/openerp-contrib-mp

« Back to merge proposal