Hi Guewen, Thanks for the review. I've integrated your remarks in the code. In the same time, the changes made to solve the dependecy problem fixes the lp:1216886 issue.
A last remark, if you run the test of 'account_statement_bankaccount_completion' with the module 'account_statement_base_import' installed in the same db, it fails (missing required value'import_type'). I don't know if it's really an issue. Regards, lmi -- https://code.launchpad.net/~acsone-openerp/banking-addons/bank-statement-reconcile-70/+merge/185047 Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~acsone-openerp/banking-addons/bank-statement-reconcile-70 into lp:banking-addons/bank-statement-reconcile-7.0. -- Mailing list: https://launchpad.net/~banking-addons-team Post to : [email protected] Unsubscribe : https://launchpad.net/~banking-addons-team More help : https://help.launchpad.net/ListHelp
