Code review comment for lp:~therp-nl/banking-addons/6.1_lp1117319_abnamro_sepa_line

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

Thanks for fixing this!

l.8,28 Maybe combine in a single dictionary of possible keys?
l.62 Looks like the last key/value pair is not stored in sepa_dict. You could copy this line after the end of the 'while' loop.

review: Needs Fixing

« Back to merge proposal