Skip to content

Conversation

@florian-dacosta
Copy link
Contributor

This module allow to create a misc accounting entry to reclassify the ecotax amounts and isolate it in a dedicated account.
Indeed, using the base ecotax module (account_ecotax), the ecotax amounts are including in the revenue/product accounts and usually you would prefer have it isolated.

Note that the module account_ecotax_tax that uses odoo taxes to manage ecotax, already has the feature of isolating the ecotax amounts using the native tax features. But like explained here : #453 there are pros and cons for this last module and in some cases, it is better to use only the first module (account_ecotax) but still beeing able to isolate the ecotax amounts.

This is the problem that this module tries to solve.

Important note : this module is NOT needed nor wanted when using account_ecotax_tax, like I just explained, so I put an exclusion in the manifest.

@florian-dacosta florian-dacosta force-pushed the 16-add-account_ecotax_accounting branch from 518db3d to 1303dca Compare March 28, 2025 08:46
@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 27, 2025
@florian-dacosta florian-dacosta force-pushed the 16-add-account_ecotax_accounting branch from 1303dca to e46086d Compare August 1, 2025 14:10
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Aug 3, 2025
@hparfr
Copy link

hparfr commented Aug 7, 2025

There is an issue with test exclusion

@florian-dacosta
Copy link
Contributor Author

There is an issue with test exclusion

Yes I know, but I have no idea on how to get around this.
The CI install all modules of the repo, I do not know if there is a way to make it different or not.

@florian-dacosta florian-dacosta force-pushed the 16-add-account_ecotax_accounting branch from 925d089 to 4f34a2d Compare August 8, 2025 10:07
This aims to avoid having a different tax depending on which localization is installed for instance
@florian-dacosta florian-dacosta force-pushed the 16-add-account_ecotax_accounting branch from 03bdca0 to 13ee5ea Compare August 8, 2025 12:26
Copy link

@hparfr hparfr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions
Copy link

github-actions bot commented Dec 7, 2025

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Dec 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants