Code review comment for lp:~pedro.baeza/openobject-addons/7.0-updated_l10n_es

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

Hi, Olivier:

Thanks for you review.

Let me try to clarify both points:

- I changed description because I copied/pasted the one from openerp-spain repository, but I didn't know that it would break RST. I can restore the original one. Can you point me to a explanation for making RST compliant texts? Another thing I take the opportunity to ask you: how can I export correctly this terms (module title, module description) and so on for translation?
- I put "noupdate" flag to 0 because I discovered some incorrect terms in account names (for example, "Inmobilizado" instead of "Inmovilizado"). As you have said, these are only templates, so it doesn't reflect in generated charts of accounts, but there are two interesting applications to renew template data: a) new generated charts of accounts have correct data; and b) with account_chart_update module, all previously generated charts of accounts can update their accounts names.

Cheers.

« Back to merge proposal