Review: Needs Fixing

Hi Erwin,

this branch includes your other proposal that adds the 
account_banking_nl_rabobank module. Can you please rebase your changes on a 
fresh branch of lp:banking-addons/6.1?

Apart from that, it *seems* like you only add the necessary field and form 
changes but I don't see where it is filled with the ClieOp keyword. Am I 
missing something?

l.33: suspicious indent but it's not your fault. I see this file already has 
tabs, not spaces. I will reformat it at the eventual merge.

-- 
https://code.launchpad.net/~endiansolutions/banking-addons/clieop_filename/+merge/141162
Your team Banking Addons Team 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