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] sale_margin_delivery: Store percentage value as into odoo #215

Closed lmignon closed 2 months ago

lmignon commented 3 months ago

Prior to this change

After

OCA-git-bot commented 3 months ago

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

lmignon commented 3 months ago

@Shide @yajo @Gelojr @fcvalgar following my comment https://github.com/oca/margin-analysis/pull/214#issuecomment-2178844145

rousseldenis commented 3 months ago

@lmignon It seems dropshipping module should be updated too.

lmignon commented 3 months ago

@lmignon It seems dropshipping module should be updated too.

it's done.

Shide commented 2 months ago

/ocabot merge nobump

Done with nobump because version is already updated manually

OCA-git-bot commented 2 months ago

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-215-by-Shide-bump-nobump, awaiting test results.

OCA-git-bot commented 2 months ago

@Shide your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-215-by-Shide-bump-nobump.

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.

Shide commented 2 months ago

/ocabot mege nobump

OCA-git-bot commented 2 months ago

Hi @Shide. Your command failed:

Invalid command: mege.

Ocabot commands

More information

Shide commented 2 months ago

/ocabot merge nobump

OCA-git-bot commented 2 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-215-by-Shide-bump-nobump, awaiting test results.

OCA-git-bot commented 2 months ago

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