Banking Addons

Merge lp:~therp-nl/banking-addons/lp1228055 into lp:banking-addons

  1. lp1228055
  2. Merge into banking-addons-70
Proposed by Holger Brunn (Therp)
Status: Merged
Merged at revision: 189
Proposed branch: lp:~therp-nl/banking-addons/lp1228055
Merge into: lp:banking-addons
Diff against target: 12 lines (+1/-1)
1 file modified
account_banking/sepa/iban.py (+1/-1)
To merge this branch: bzr merge lp:~therp-nl/banking-addons/lp1228055
Reviewer Review Type Date Requested Status
Stefan Rijnhart (Opener) Approve
Pedro Manuel Baeza code review, no test Approve
Review via email: mp+186743@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

I don't know about Austrian bank format, but I suppose that you must be right.

Regards.

review: Approve (code review, no test)
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

LGTM!

review: Approve
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Merged with 7.0 and 6.1 (cherrypicked)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account_banking/sepa/iban.py'
2--- account_banking/sepa/iban.py 2013-04-15 13:59:50 +0000
3+++ account_banking/sepa/iban.py 2013-09-20 10:32:20 +0000
4@@ -190,7 +190,7 @@
5 BBAN_formats = {
6 'AL': BBANFormat('CCBBBBVAAAAAAAAAAAAAAAAAA', '%B%A'),
7 'AD': BBANFormat('CCCCBBBBAAAAAAAAAAAA', '%A'),
8- 'AT': BBANFormat('BBBBBAAAAAAAAAAA', '%A BLZ %C'),
9+ 'AT': BBANFormat('BBBBBAAAAAAAAAAA', '%A BLZ %B'),
10 'BE': BBANFormat('CCCAAAAAAAVV', '%C-%A-%V'),
11 'BA': BBANFormat('BBBCCCAAAAAAAA', '%I'),
12 'BG': BBANFormat('BBBBCCCCAAAAAAAAAA', '%I'),

Subscribers

People subscribed via source and target branches

to status/vote changes: