openerp-community-reviewer team mailing list archive
-
openerp-community-reviewer team
-
Mailing list archive
-
Message #03943
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