Code review comment for lp:~tsy/openobject-addons/l10n_syscohada

Revision history for this message
Mustufa Rangwala (Open ERP) (mra-tinyerp) wrote :

to be fix/check:

1. fr_FR.po has some problem.

/addons/trunk$ msgfmt l10n_syscohada/i18n/fr_FR.po
l10n_syscohada/i18n/fr_FR.po:68: duplicate message definition...
l10n_syscohada/i18n/fr_FR.po:64: ...this is the location of the first definition
msgfmt: found 1 fatal error

2. for account.account.type => You should have report_type (P&L / BS Category) this field is newly create on account.account.type object. So according value related accounts will be display on respective reports (Balance Sheet Profit & Loss Account.

3. you have defined accuont.tax.code.tamplate so that i can not see Accounting/Charts/Chart of Taxes ..I am not sure about this..

Thank you,
mra

« Back to merge proposal