search

Vauxoo / addons-vauxoo

All our modules related to developments that solves generic issues on Odoo, or that solve internal problems on Odoo Core, if something is here, maybe it is solving an issue in your company, try it and report what you see.
http://www.vauxoo.com
193 stars 288 forks source link

[8.0] Fix subtotal considering taxes with tax included in price #1214

Closed alan196 closed 7 years ago

alan196 commented 7 years ago

This PR fix an issue in the module invoice_discount that was generated when you generate an invoice with taxes that is configured with tax included in price. When you generate an invoice in odoo-mexico-v2 the subtotal and total were the same.

alan196 commented 7 years ago

@nhomar @moylop260 @luistorresm can you review this please 🙏

alan196 commented 7 years ago

@luistorresm Change is done!

luistorresm commented 7 years ago

@alan196

Could you add a dummy PR to odoo-mexico-v2?

Regards

alan196 commented 7 years ago

https://github.com/Vauxoo/odoo-mexico-v2/pull/479

alan196 commented 7 years ago

@luistorresm Dummy PR in odoo-mexico-v2 Runbot is green. Travis fails in LINT but is green in Odoo test.

luistorresm commented 7 years ago

@JulioSerna

What do you think?

Regards