← Back to team overview

banking-addons-team team mailing list archive

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