OCA / margin-analysis

Odoo Financial Controlling (margin, costs, ...)
https://odoo-community.org/project/9
GNU Affero General Public License v3.0
74 stars 161 forks source link

[13.0][FIX] sale_margin_delivered: non storable #150

Closed chienandalu closed 1 year ago

chienandalu commented 2 years ago

Fw of:

We don't want to compute margin based on stock moves for non storable products. This specially true for services and kits.

TT39817

please review @sergio-teruel @CarlosRoca13

OCA-git-bot commented 2 years ago

Hi @sergio-teruel, some modules you are maintaining are being modified, check this out!

pedrobaeza commented 1 year ago

/ocabot merge patch

Please continue with the fw-port chain

OCA-git-bot commented 1 year ago

On my way to merge this fine PR! Prepared branch 13.0-ocabot-merge-pr-150-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 13.0-ocabot-merge-pr-150-by-pedrobaeza-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

pedrobaeza commented 1 year ago

/ocabot merge patch

OCA-git-bot commented 1 year ago

On my way to merge this fine PR! Prepared branch 13.0-ocabot-merge-pr-150-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

Congratulations, your PR was merged at 4241452ddd15ba1df76f0375e74ef519a70e81a5. Thanks a lot for contributing to OCA. ❤️