Skip to content

Conversation

@fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Mar 17, 2025

When manually adding a byproduct to a MO, the warehouse is not set because default_warehouse_id is missing from the context. This means that the push move from post-prod to stock is never created and the byproduct is stuck in post-production location. This fix adds the context key in the MO form view.

Steps:

  • Activate routes & by-product in the main settings
  • Activate manufacturing in 3 steps on the warehouse
  • Create manufacturing order for a product and add a by-product with a positive quantity
  • Confirm manufacturing order
  • Pick components
  • Produce & close MO

opw-4620008


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Forward-Port-Of: #201619

@robodoo
Copy link
Contributor

robodoo commented Mar 17, 2025

Pull request status dashboard

@fw-bot
Copy link
Contributor Author

fw-bot commented Mar 17, 2025

@Sehnde @clesgow cherrypicking of pull request #201619 failed.

stdout:

Auto-merging addons/mrp/tests/test_byproduct.py
CONFLICT (content): Merge conflict in addons/mrp/tests/test_byproduct.py
Auto-merging addons/mrp/views/mrp_production_views.xml
CONFLICT (content): Merge conflict in addons/mrp/views/mrp_production_views.xml

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

In the former case, you may want to edit this PR message as well.

⚠️ after resolving this conflict, you will need to merge it via @robodoo.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@robodoo robodoo added forwardport This PR was created by @fw-bot conflict There was an error while creating this forward-port PR labels Mar 17, 2025
@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Mar 17, 2025
@Sehnde Sehnde force-pushed the 17.0-16.0-opw-4620008-mrp-3-steps-byproduct-snd-422651-fw branch 2 times, most recently from f7e57ff to fec6e79 Compare March 17, 2025 12:47
When manually adding a byproduct to a MO, the warehouse is not set
because `default_warehouse_id` is missing from the context.
This means that the push move from post-prod to stock is never
created and the byproduct is stuck in post-production location.
This fix adds the context key in the MO form view.

Steps:
- Activate routes & by-product in the main settings
- Activate manufacturing in 3 steps on the warehouse
- Create manufacturing order for a product and add a by-product with a positive quantity
- Confirm manufacturing order
- Pick components
- Produce & close MO

opw-4620008

X-original-commit: 2adabe7
@Sehnde Sehnde force-pushed the 17.0-16.0-opw-4620008-mrp-3-steps-byproduct-snd-422651-fw branch from fec6e79 to 59e1319 Compare March 17, 2025 14:13
@Sehnde
Copy link
Contributor

Sehnde commented Mar 17, 2025

robodoo r+

robodoo pushed a commit that referenced this pull request Mar 17, 2025
When manually adding a byproduct to a MO, the warehouse is not set
because `default_warehouse_id` is missing from the context.
This means that the push move from post-prod to stock is never
created and the byproduct is stuck in post-production location.
This fix adds the context key in the MO form view.

Steps:
- Activate routes & by-product in the main settings
- Activate manufacturing in 3 steps on the warehouse
- Create manufacturing order for a product and add a by-product with a positive quantity
- Confirm manufacturing order
- Pick components
- Produce & close MO

opw-4620008

closes #202054

X-original-commit: 2adabe7
Signed-off-by: Quentin Wolfs (quwo) <quwo@odoo.com>
Signed-off-by: Stéphane Diez (snd) <snd@odoo.com>
@robodoo robodoo closed this Mar 17, 2025
@fw-bot fw-bot deleted the 17.0-16.0-opw-4620008-mrp-3-steps-byproduct-snd-422651-fw branch March 24, 2025 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflict There was an error while creating this forward-port PR forwardport This PR was created by @fw-bot OE the report is linked to a support ticket (opw-...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants