Customer supplier invoice on dropshipping

[lepistone]

fixes odoo/odoo#4131 with a separate addon

[pedrobaeza]

You should use OCA template in https://github.com/OCA/maintainers-tools/blob/master/template/module/README.rst

[pedrobaeza]

Thanks for this very needed feature!

I see as a better name stock_dropshipping_dual_invoice, to follow conventions and short it. What do you think?

Please check Travis error also.

[lepistone]

Thanks as usual for reviewing @pedrobaeza!

The red build is due to stock_route_transit changing the default dropshipping route. Before just isolating the two modules, I'll take a moment to discuss about that with @gurneyalex.

For the title I agree dropshipping_dual_invoice is clearer and shorter, thanks! However I wonder: do we really need to repeat stock_ everywhere? This module, in fact, interacts with sales, purchases, stock and accounting. Moreover, all those modules are already grouped in a repository dealing with stock.

[pedrobaeza]

Yeah, I agree that we can omit stock_ part, but I put it because the module that starts all of this is called stock_dropshipping, and this way, all related modules are seen together in all lists (module list in Odoo, repository list, and so on).

[coveralls]

Coverage increased (+3.4%) when pulling 274971a on lepistone:customer_supplier_invoice_on_dropshipping into b30ef17 on OCA:8.0.

[pedrobaeza]

Thanks!

👍

[coveralls]

Coverage increased (+5.36%) when pulling 29f1f1b on lepistone:customer_supplier_invoice_on_dropshipping into 1e7938c on OCA:8.0.

[coveralls]

Coverage increased (+5.36%) when pulling 29f1f1b on lepistone:customer_supplier_invoice_on_dropshipping into 1e7938c on OCA:8.0.

[coveralls]

Coverage increased (+5.09%) when pulling d3681a3 on lepistone:customer_supplier_invoice_on_dropshipping into 1e7938c on OCA:8.0.

[gurneyalex]

👍