Banking Addons

Code review comment for lp:~therp-nl/banking-addons/ba7.0-MIG-import

  1. ba7.0-MIG-import
  2. Merge into banking-addons-70
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Hi Alexandre,

totally agreed. I refactored out this legacy wildcard import.

AB stands for Account Banking, the name of the original project. It was used to mark custom lines in the create_move_from_st_line() method that it replaced in earlier versions. Of course, this version takes advantage of the refactorings in the original method driven by Camp2camp (thanks!) so that it can apply proper inheritance. I only forgot to update the comments.

« Back to merge proposal