account-analytic
account-analytic copied to clipboard
[14.0] product_analytic_sale
The readme gives an example of a scenario where this module product_analytic_sale is required.
@alexis-via I just found a bug in case the Sale Order Line as no product_id (type note). I added a commit to fix it. cc @rvalyi
It should be great to add a little test.
This works +1
As @rousseldenis , it could be great a little test #462 (comment)
My curiosity / question is , why not improve product_analytic with this so we don't need glue modules? I complicated so we need another module
@rafaelbn This to avoid a strong dependency on sale in product_analytic
/ocabot merge nobump
This PR looks fantastic, let's merge it! Prepared branch 14.0-ocabot-merge-pr-462-by-dreispt-bump-nobump, awaiting test results.
Congratulations, your PR was merged at 3b0ffc5564d7fa8a4ad40fc5a2f7d484e12202cd. Thanks a lot for contributing to OCA. ❤️