Banking Addons

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

  1. 6.1-link_partner_wizard
  2. Merge into 6.1
Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

l.183 I think the intended tuple of types was (int, long) instead of (int, float).

l.182 an indentation level could be avoided by returning early in the method (maybe a matter of taste though)

Besides that, LGTM

review: Needs Fixing (code review, no test)

« Back to merge proposal