Review: Resubmit Hi again,
Here is a new version. I adressed all of Stefan's comments: - module name - base_iban dependency - migration of manual_bank_transfer xml id - the trick/hack/thing about making line2bank receive the payment mode :) @Stefan, have you ever attempted to integrate that last one in a MP to the core? The fix there would be simple, although difficult to explain in a bug report. plus: - a bit of pep8 - a view bug I discovered while testing -- 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
