Merge lp:~therp-nl/banking-addons/ba61-abnamro_intl_iban into lp:banking-addons/6.1

Proposed by Stefan Rijnhart (Opener)
Status: Merged
Merged at revision: 167
Proposed branch: lp:~therp-nl/banking-addons/ba61-abnamro_intl_iban
Merge into: lp:banking-addons/6.1
Diff against target: 12 lines (+1/-1)
1 file modified
account_banking_nl_abnamro/abnamro.py (+1/-1)
To merge this branch: bzr merge lp:~therp-nl/banking-addons/ba61-abnamro_intl_iban
Reviewer Review Type Date Requested Status
Guewen Baconnier @ Camptocamp Approve
Ronald Portier (Therp) Approve
Review via email: mp+161094@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Ronald Portier (Therp) (rportier1962) wrote :

LGTM. No test.

review: Approve
Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'account_banking_nl_abnamro/abnamro.py'
--- account_banking_nl_abnamro/abnamro.py 2013-04-17 14:24:33 +0000
+++ account_banking_nl_abnamro/abnamro.py 2013-04-26 11:28:31 +0000
@@ -294,7 +294,7 @@
294 if fields[2].startswith('/'):294 if fields[2].startswith('/'):
295 self.remote_account = fields[2][1:].strip()295 self.remote_account = fields[2][1:].strip()
296 else:296 else:
297 self.message += ' ' + fields[2].strip()297 self.remote_account = fields[2].strip()
298 # fourth column contains remote owner298 # fourth column contains remote owner
299 self.remote_owner = (len(fields) > 3 and fields[3].strip() or299 self.remote_owner = (len(fields) > 3 and fields[3].strip() or
300 False)300 False)

Subscribers

People subscribed via source and target branches