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

[17.0][MIG] mrp_multi_level_estimate #1273

Merged
merged 45 commits into from
May 21, 2024

Conversation

JasminSForgeFlow
Copy link
Contributor

Standard Migration

@ForgeFlow

LoisRForgeFlow and others added 30 commits May 21, 2024 08:26
when group days of estimates is greater than the lengh of the date
range in the estimate, the estimate is overstated and can generate
unexpected result. Enhance help message to highligh this and
prevent it from happening.
Rounding should be done at the end of the calculation, using the daily
quantity already rounded can lead to errors.
… range.

Do not overstate demand, a test hightlights the issue being solved.
Currently translated at 100.0% (5 of 5 strings)

Translation: manufacture-15.0/manufacture-15.0-mrp_multi_level_estimate
Translate-URL: https://translation.odoo-community.org/projects/manufacture-15-0/manufacture-15-0-mrp_multi_level_estimate/it/
…rategy

Introduces a new concept to define the strategy to apply when estimates
coexist with other sources of demand.
Remove unneeded dependency to date_range and relicense to LGPL.
weblate and others added 15 commits May 21, 2024 08:26
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: manufacture-15.0/manufacture-15.0-mrp_multi_level_estimate
Translate-URL: https://translation.odoo-community.org/projects/manufacture-15-0/manufacture-15-0-mrp_multi_level_estimate/
Currently translated at 100.0% (11 of 11 strings)

Translation: manufacture-15.0/manufacture-15.0-mrp_multi_level_estimate
Translate-URL: https://translation.odoo-community.org/projects/manufacture-15-0/manufacture-15-0-mrp_multi_level_estimate/it/
Currently translated at 100.0% (11 of 11 strings)

Translation: manufacture-16.0/manufacture-16.0-mrp_multi_level_estimate
Translate-URL: https://translation.odoo-community.org/projects/manufacture-16-0/manufacture-16-0-mrp_multi_level_estimate/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: manufacture-16.0/manufacture-16.0-mrp_multi_level_estimate
Translate-URL: https://translation.odoo-community.org/projects/manufacture-16-0/manufacture-16-0-mrp_multi_level_estimate/
…considered

When we consider forecasts as the 'Indirect' demand, we should always keep considering the demand coming from Manufacturing Orders, as they are 'Direct' demand.
@LoisRForgeFlow
Copy link
Contributor

/ocabot migration mrp_multi_level_estimate

Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

Functional and technical review, all good 👍

@LoisRForgeFlow
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone May 21, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request May 21, 2024
19 tasks
@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 99f20e9 into OCA:17.0 May 21, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@LoisRForgeFlow LoisRForgeFlow deleted the 17.0-mig-mrp_multi_level_estimate branch May 21, 2024 08:25
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.