Review: Approve

Seems correct, but i didn't run it or tried with a real instance. I guess it's 
good to be merged if you did so.

as discussed, it should be a good idea to also include: 
- a check that raises an error if the statement's journal and its default 
accounts don't share the same configuration for secondary currency.
- a domain in the account.move.line m2m of bank statements when going by 
"periodical processing \ bank reconciliation \ .." in order to make sure that 
people encode there only accounting entries with a correct secondary currency 
set.

Thanks,
Quentin
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/5.0-opw-7285-pso/+merge/76019
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/5.0-opw-7285-pso.

_______________________________________________
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