Review: Approve

Some remarks:
* We should not discard the account installer ('Install your chart of 
accounts'), but instead reuse it.  Note that it must be launched automatically 
when you install the module account.
* The fields 'default_paypal_account' and 'company_footer' should be related 
fields (through company_id).
* The fields 'sale_journal_id', 'sale_refund_journal_id', 
'purchase_journal_id', 'purchase_refund_journal_id' and their corresponding 
related fields depend on the value of company_id.  They must be updated when 
company_id is changed!
* Same for 'fiscalyear_id'.

I will do the necessary fixes once the branch is merged.

Thanks for the work!
Raphael

-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-v62_config_account-uco/+merge/98995
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-v62_config.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to