← Back to team overview

openerp-community team mailing list archive

Re: [Merge] lp:~tsy/openobject-addons/l10n_syscohada into lp:openobject-addons

 

Review: Needs Fixing
Hi Tidiane Sy (BAAMTU),

i've checked and i can see that you did really a good job. 

The only very small thing that may need to be fixed is the internal type of the account "521100 Banque X" that should be a view account. Indeed, the wizard will create children accounts for it (for deposit, current...), and we think that it's a best practice to allow having children only for account of type view, otherwise it may mess the computation, analyses and what so forth. 

So for this reason, there is a constraint currently in the accounts that forbid such a configuration, and you should change accordingly to it, otherwise it will raise as soon as you try to modify and save this account.

Thanks for your contribution, again. Let us know as soon as you made this modification in your branch.

Quentin.
-- 
https://code.launchpad.net/~tsy/openobject-addons/l10n_syscohada/+merge/58998
Your team OpenERP Community is subscribed to branch lp:~tsy/openobject-addons/l10n_syscohada.


References