Review: Needs Information

Ciao Sergio,

this is very similar to
https://code.launchpad.net/~nicolariolini/account-invoicing/add_module_account_payment_term_month/+merge/212400

I slightly prefer the Nicola's module, just because it use an explicit field 
'months' and thus it is a little more clear.

But your module has tests so it got more points :-) But the tests can be easily 
adapted to account_payment_term_month

What do you think?
-- 
https://code.launchpad.net/~icsergio/account-payment/commercial-year-payment/+merge/210700
Your team Account Payment is requested to review the proposed merge of 
lp:~icsergio/account-payment/commercial-year-payment into lp:account-payment.

-- 
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