banking-addons-team team mailing list archive
-
banking-addons-team team
-
Mailing list archive
-
Message #00991
Re: [Merge] lp:~therp-nl/banking-addons/7.0-add_camt_import into lp:banking-addons
Review: Approve
Just two minor things:
#25 to make this an even more valuable utility function, consider making tail_length a keyword argument. And another for maximum length maybe
#31, #218 what is this alias good for? (I think it degrades readability)
--
https://code.launchpad.net/~therp-nl/banking-addons/7.0-add_camt_import/+merge/190638
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons.
References