[19.0][MIG] ddmrp#600
Merged
Merged
Conversation
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: ddmrp-15.0/ddmrp-15.0-ddmrp Translate-URL: https://translation.odoo-community.org/projects/ddmrp-15-0/ddmrp-15-0-ddmrp/
…s set The onchange only has a visual effect. Actual changes to adu must be made in the create and write methods.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: ddmrp-15.0/ddmrp-15.0-ddmrp Translate-URL: https://translation.odoo-community.org/projects/ddmrp-15-0/ddmrp-15-0-ddmrp/
remove unneded attributes on related field definition.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: ddmrp-15.0/ddmrp-15.0-ddmrp Translate-URL: https://translation.odoo-community.org/projects/ddmrp-15-0/ddmrp-15-0-ddmrp/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: ddmrp-15.0/ddmrp-15.0-ddmrp Translate-URL: https://translation.odoo-community.org/projects/ddmrp-15-0/ddmrp-15-0-ddmrp/
Currently translated at 100.0% (291 of 291 strings) Translation: ddmrp-15.0/ddmrp-15.0-ddmrp Translate-URL: https://translation.odoo-community.org/projects/ddmrp-15-0/ddmrp-15-0-ddmrp/es_AR/
A hook is added to get production delay because subcontracting BoMs calculate production time differently, from the time of the first vendor with subcontracted type.
- Add link to Stock Demand Estimates from Stock Buffer - Add link to MRP BoM from Stock Buffer - Add link to Stock Buffers from Product Variants - Add link to Stock Buffer from MRP BoM Line
- On the top, we will have contextualization about the bufffer. - On the left side, we will have settings and info about the configuration of the buffer and procurements. - On the right side, we will have information about the buffer. Apart from this, you will be able to access the source of ADU, Qualified demand and Incoming (DLT) calculations.
PaulaMForgeFlow
force-pushed
the
19.0-mig-ddmrp
branch
from
1ca894c to
fd7fe5c
Compare
Contributor
PaulaMForgeFlow
force-pushed
the
19.0-mig-ddmrp
branch
from
6d1d0ec to
324f453
Compare
Contributor
|
Cherry picked a recent change from 18.0 #606 |
LoisRForgeFlow
requested changes
Apr 24, 2026
Contributor
LoisRForgeFlow
left a comment
LoisRForgeFlow
left a comment
There was a problem hiding this comment.
Code review (see comments below) and quick functional review, main issues related to UoM migration.
PaulaMForgeFlow
force-pushed
the
19.0-mig-ddmrp
branch
from
dffedad to
01fc232
Compare
LoisRForgeFlow
requested changes
Apr 28, 2026
Contributor
LoisRForgeFlow
left a comment
LoisRForgeFlow
left a comment
There was a problem hiding this comment.
Thank for attending my comments, I did a new review:
Comment on lines
+1405
to
+1406
| if not self.env.company.ddmrp_adu_calc_include_scrap: | ||
| domain.append(("location_id.scrap_location", "=", False)) | ||
| domain.append(("location_id.usage", "!=", "inventory")) |
Contributor
There was a problem hiding this comment.
@PaulaMForgeFlow Then in order to keep the setting meaning, you need to identify scraps in a different way. I suggest using scrap_id not set.
PaulaMForgeFlow
force-pushed
the
19.0-mig-ddmrp
branch
from
01fc232 to
f107da8
Compare
Show only indirect demand button when the method is going to use it.
The moves shown were not all qualified demand due to an issue in the date filtering. In addition to fixing this issue: - We show all demand moves but filtered by default to the ones that are qualified demand. - Added the reserved quantity to ease understanding that not all quantity is considered.
PaulaMForgeFlow
force-pushed
the
19.0-mig-ddmrp
branch
from
f107da8 to
186d8b6
Compare
LoisRForgeFlow
approved these changes
Apr 29, 2026
Contributor
LoisRForgeFlow
left a comment
LoisRForgeFlow
left a comment
There was a problem hiding this comment.
All comments attended and resolved, functional review LGTM 👍
Contributor
|
/ocabot migration ddmrp |
Contributor
|
Time to merge! /ocabot merge nobump |
Contributor
|
Hey, thanks for contributing! Proceeding to merge this for you. |
Contributor
|
Congratulations, your PR was merged at 19720c4. Thanks a lot for contributing to OCA. ❤️ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Standard Migration