search

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

[16.0][FW-PORT][IMP] sale_margin_security: Avoid display margin in measures pivot view. #190

Closed ValentinVinagre closed 10 months ago

ValentinVinagre commented 10 months ago

Forward-port of https://github.com/OCA/margin-analysis/pull/188

Set group_sale_margin_security in all sensible fields.

OCA-git-bot commented 10 months ago

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

ValentinVinagre commented 10 months ago

@HaraldPanten

pedrobaeza commented 10 months ago

Please mention the PR that this is a fw-port of. This helps to speed up reviews.

ValentinVinagre commented 10 months ago

Please mention the PR that this is a fw-port of. This helps to speed up reviews.

done 😄

OCA-git-bot commented 10 months ago

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-190-by-pedrobaeza-bump-major, awaiting test results.

ValentinVinagre commented 10 months ago

I was talking about putting this:

Selección_022

but I have already done it.

/ocabot merge major

For the next few times I will keep it in mind, I didn't know how it was managed :) thanks Pedro.

OCA-git-bot commented 10 months ago

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