Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[14.0][MIG] mrp_production_quant_manual_assign #662

Merged
merged 6 commits into from
Sep 3, 2021

Conversation

LoisRForgeFlow
Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow commented Jul 26, 2021

@LoisRForgeFlow LoisRForgeFlow force-pushed the 14.0-mrp_quant_manual_assign branch 2 times, most recently from 861b8da to 399f6ff Compare July 27, 2021 10:34
Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functional review on local 👍 Runbot warning seems unrelated:
2021-07-27 10:54:08,734 169 WARNING openerp_test odoo.models: @onchange('standard_price', 'activity_cost_ids', 'activity_cost_ids.standard_price') parameters must be field names
Can you check?

Copy link

@AdriaGForgeFlow AdriaGForgeFlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functional review 👍

@LoisRForgeFlow
Copy link
Contributor Author

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-662-by-LoisRForgeFlow-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Sep 1, 2021
Signed-off-by LoisRForgeFlow
@OCA-git-bot
Copy link
Contributor

@LoisRForgeFlow your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-662-by-LoisRForgeFlow-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.

@LoisRForgeFlow
Copy link
Contributor Author

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-662-by-LoisRForgeFlow-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 263a11d into OCA:14.0 Sep 3, 2021
@OCA-git-bot
Copy link
Contributor

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

@LoisRForgeFlow LoisRForgeFlow deleted the 14.0-mrp_quant_manual_assign branch September 3, 2021 14:37
enriquemartin pushed a commit to Digital5-Odoo/manufacture that referenced this pull request Oct 1, 2021
…les * Trigger line instead of test and user
stupidgoats pushed a commit to stupidgoats/manufacture that referenced this pull request Oct 1, 2021
…les * Trigger line instead of test and user
hpatelserpentcs pushed a commit to hpatelserpentcs/manufacture that referenced this pull request Dec 29, 2021
…les * Trigger line instead of test and user
mymage pushed a commit to mymage/manufacture that referenced this pull request Dec 10, 2022
…les * Trigger line instead of test and user
mymage pushed a commit to mymage/manufacture that referenced this pull request Dec 26, 2022
…les * Trigger line instead of test and user
mymage pushed a commit to mymage/manufacture that referenced this pull request Jan 17, 2023
…les * Trigger line instead of test and user
peluko00 pushed a commit to APSL/manufacture that referenced this pull request May 14, 2024
…les * Trigger line instead of test and user
Trivedi-Vacha-SerpentCS pushed a commit to Trivedi-Vacha-SerpentCS/manufacture that referenced this pull request Oct 23, 2024
…les * Trigger line instead of test and user
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants