Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16][REF] account_ecotax split #454

Open
wants to merge 15 commits into
base: 16.0
Choose a base branch
from

Commits on Oct 29, 2024

  1. Configuration menu
    Copy the full SHA
    f4d842b View commit details
    Browse the repository at this point in the history
  2. misc-fix2review

    bealdav authored and mourad-ehm committed Oct 29, 2024
    Configuration menu
    Copy the full SHA
    577fc44 View commit details
    Browse the repository at this point in the history
  3. Apply suggestions from code review

    Co-authored-by: Maksym Yankin <[email protected]>
    mourad-ehm and yankinmax committed Oct 29, 2024
    Configuration menu
    Copy the full SHA
    0cfeb6f View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    d30c342 View commit details
    Browse the repository at this point in the history
  5. [16.0] FIX module version

    mourad-ehm committed Oct 29, 2024
    Configuration menu
    Copy the full SHA
    0ce73c2 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    faa8a0c View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    758dfc8 View commit details
    Browse the repository at this point in the history
  8. Update account_ecotax/models/account_ecotax_classification.py

    Co-authored-by: Alexandre Fayolle <[email protected]>
    mourad-ehm and gurneyalex committed Oct 29, 2024
    Configuration menu
    Copy the full SHA
    ce8ad37 View commit details
    Browse the repository at this point in the history
  9. Apply suggestions from code review

    Co-authored-by: Alexandre Fayolle <[email protected]>
    mourad-ehm and gurneyalex committed Oct 29, 2024
    Configuration menu
    Copy the full SHA
    563aad6 View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    ed92a1c View commit details
    Browse the repository at this point in the history
  11. [FIX] account_ecotax: test fixes

    rvalyi authored and mourad-ehm committed Oct 29, 2024
    Configuration menu
    Copy the full SHA
    ec38154 View commit details
    Browse the repository at this point in the history
  12. IMP readme usage

    mourad-ehm committed Oct 29, 2024
    Configuration menu
    Copy the full SHA
    fc67913 View commit details
    Browse the repository at this point in the history

Commits on Oct 31, 2024

  1. FIX tests

    mourad-ehm committed Oct 31, 2024
    Configuration menu
    Copy the full SHA
    ff0cdc3 View commit details
    Browse the repository at this point in the history

Commits on Nov 18, 2024

  1. [16][REF] split account_ecotax into account_ecotax and account_ecotax…

    …_tax
    
    The goal is to be able to choose between the implementation with and without using the odoo tax mechanism.
    The advantages of the implementation based on Odoo tax mechanims are :
    - Possibility to choose if product price include or exclude the ecotax amounts
    - Isolate the ecotax amounts into a specifc accounting account
    The disadvantage is that it adds a small layer of complexity and you have to manage the tax configuration and see all those ecotax taxes on your invoices
    And a major difference which can be good or not depending on your use cases, the ecotax amounts are not in the turnover when using Odoo tax mechanism
    florian-dacosta committed Nov 18, 2024
    Configuration menu
    Copy the full SHA
    8bc3f4a View commit details
    Browse the repository at this point in the history

Commits on Nov 20, 2024

  1. [IMP] account_ecotax : remove dependency on AccountTestInvoicingCommo…

    …n from tests
    
    The dependency does not really ease the present test but forces us to put the test as post-installed which cant work as account_ecotax_tax changes the account_ecotax behavior
    Instead of isolating the tests of account_ecotax, it seems better to get rid of the AccountTestInvoicingCommon dependency. It also speed up the tests.
    florian-dacosta committed Nov 20, 2024
    Configuration menu
    Copy the full SHA
    7e230c6 View commit details
    Browse the repository at this point in the history