Code review comment for lp:~yannick-buron/sale-wkfl/sale-wkfl

Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

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.

review: Disapprove

« Back to merge proposal