On 05/24/2013 07:12 PM, Stefan Rijnhart (Therp) wrote:
> 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?

To be sure this module does not influence any calculation or move line 
creation :-)

Anyway, I can remove some of them, if unnecessary.


> Nit: l.2866 could be kept at < 80 chars
>

Done


-- 
https://code.launchpad.net/~elbati/account-payment/adding_account_voucher_supplier_invoice_number_7/+merge/165622
Your team Account Payment is subscribed to branch lp:account-payment/7.0.

-- 
Mailing list: https://launchpad.net/~account-payment-team
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~account-payment-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to