Review: Needs Fixing

Stéphane,

once again thank you for your patience. I have succesfully tested migrating 
existing databases to the current state of the branch. Meanwhile, I am writing 
a lengthy test that simulates a payment order - bank statement round trip, but 
that should not keep back the merge of this branch.

In the last revisions, a slight error showed up: BaseModel._auto_init() 
actually returns something that can be meaningful. It does not seem to make a 
difference here but I think it would be best to return the result of super() 
after line 1226.

And then we're done.

-- 
https://code.launchpad.net/~acsone-openerp/banking-addons/ba-70-payment-export-refactoring/+merge/179543
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons.

-- 
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

Reply via email to