[16.0][FIX] sale_margin_delivered_security: Add security groups to right fi…

OCA / margin-analysis

Odoo Financial Controlling (margin, costs, ...)

https://odoo-community.org/project/9

GNU Affero General Public License v3.0

71 stars 161 forks source link

[16.0][FIX] sale_margin_delivered_security: Add security groups to right fi… #208

Closed lmignon closed 3 months ago

lmignon commented 3 months ago

…elds

The fields to secure are defined on the sale.order.line model not on sale.order

OCA-git-bot commented 3 months ago

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

yajo commented 3 months ago

Please @rafaelbn @gelojr @fcvalgar do a functional review.

rafaelbn commented 3 months ago

@moduon MT-5158 Functional team will review today, thank you! 😄

OCA-git-bot commented 3 months ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

lmignon commented 3 months ago

@OCA/accounting-maintainers someone to merge this one plz?

pedrobaeza commented 3 months ago

/ocabot merge patch

OCA-git-bot commented 3 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-208-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 3 months ago

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