← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~yannick-buron/sale-wkfl/sale-wkfl into lp:sale-wkfl

 

Review: Disapprove

Hi, Yannick, I think that your functionality overlaps the one of the sale_order_partial_invoice module that is on account-invoicing repository. It is there because it can be considered as an "invoicing" action. I think that it's more suitable to improve this module, if there is room for it, instead of providing another one with a similar approach.

If I am wrong, please tell me. Until then, I'm going to set this as Disapprove.

Regards.
-- 
https://code.launchpad.net/~yannick-buron/sale-wkfl/sale-wkfl/+merge/207468
Your team Sale Core Editors is subscribed to branch lp:sale-wkfl.


References