Code review comment for lp:~elbati/account-payment/adding_account_voucher_supplier_invoice_number_7
Revision history for this message
| Stefan Rijnhart (Opener) (stefan-opener) wrote | # |
review:
Needs Information
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
Looks good, functionally. I am just wondering, why did you include the whole test suite of the voucher module, given that this module hardly influences any calculation or move line creation?
Nit: l.2866 could be kept at < 80 chars