OCA / account-fiscal-rule

Odoo Taxes & Fiscal Rules Management
GNU Affero General Public License v3.0
58 stars 182 forks source link

[16][REF] account_ecotax split #454

Open florian-dacosta opened 1 week ago

florian-dacosta commented 1 week ago

This PR aims to replace this one : https://github.com/OCA/account-fiscal-rule/pull/428 as explained here : https://github.com/OCA/account-fiscal-rule/issues/453

Going from the module here : https://github.com/OCA/account-fiscal-rule/pull/428 to account_ecotax_tax of this PR does not require migration script, just the installation of the new account_ecotax_tax module should be enough.

florian-dacosta commented 2 days ago

I think the PR is ready, reviews are very welcome @gurneyalex @mourad-ehm @hparfr @bealdav @yankinmax @epanisset @PaulGoubert

I've added basic override of the invoice report in account_ecotax to display the ecotax at line level and invoice level. I've done the same in account_ecotax_tax, but the result is a bit worse because as the ecotax is a tax, it is harder to override this part. This part can be improved by whoever needs it I guess.