[17.0][MIG] mrp_production_serial_matrix #1443
Merged
Conversation
This file contains 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
DavidJForgeFlow
force-pushed
the
17.0-mig-mrp_production_serial_matrix
branch
from
32687fa to
f9a0347
Compare
|
There is an error with the web_widget_x2many_2d_matrix, seems to be deleting the domain applied to the lots (the ones that checks the allowed lots). I think this could be fixed after migration as except of the domain of the lot (which is also checked at validation) all the rest seems to be working fine. |
DavidJForgeFlow
force-pushed
the
17.0-mig-mrp_production_serial_matrix
branch
from
f9a0347 to
3845829
Compare
JasminSForgeFlow
force-pushed
the
17.0-mig-mrp_production_serial_matrix
branch
2 times, most recently
from
b544a3c to
997d480
Compare
|
Issue fixed in OCA/web#3093. |
LoisRForgeFlow
requested changes
Feb 24, 2025
There was a problem hiding this comment.
I got the following error while testing including lots:
button_validate
self._consume_selected_lots(move, matrix_lines)
File "/opt/odoo/auto/addons/mrp_production_serial_matrix/wizards/mrp_production_serial_matrix.py", line 335, in _consume_selected_lots
elif float_is_zero(ml.product_qty, precision_digits=precision_digits):
AttributeError: 'stock.move.line' object has no attribute 'product_qty'
997d480 to
17892cc
Compare
Fixed now! |
LoisRForgeFlow
approved these changes
Feb 24, 2025
mrp_production_serial_matrix/wizards/mrp_production_serial_matrix_line.py
Outdated
Show resolved
Hide resolved
DavidJForgeFlow
force-pushed
the
17.0-mig-mrp_production_serial_matrix
branch
from
17892cc to
cfd5a51
Compare
mrp_production_serial_matrix/wizards/mrp_production_serial_matrix_view.xml
Outdated
Show resolved
Hide resolved
User should be able to use the matrix when the MO is reserved and select the lots that are available plus the ones reserved specifically for the MO.
If the component lines are changed they might differ from what it is stated in the BoM. It is better to rely on compoent lines to generate Matrix. New option to decide whether to add or not products tracked by lots to the matrix is added. Product tracked by serials are always added.
Currently translated at 100.0% (42 of 42 strings) Translation: manufacture-14.0/manufacture-14.0-mrp_production_serial_matrix Translate-URL: https://translation.odoo-community.org/projects/manufacture-14-0/manufacture-14-0-mrp_production_serial_matrix/it/
Currently translated at 100.0% (42 of 42 strings) Translation: manufacture-14.0/manufacture-14.0-mrp_production_serial_matrix Translate-URL: https://translation.odoo-community.org/projects/manufacture-14-0/manufacture-14-0-mrp_production_serial_matrix/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: manufacture-15.0/manufacture-15.0-mrp_production_serial_matrix Translate-URL: https://translation.odoo-community.org/projects/manufacture-15-0/manufacture-15-0-mrp_production_serial_matrix/
Currently translated at 100.0% (42 of 42 strings) Translation: manufacture-15.0/manufacture-15.0-mrp_production_serial_matrix Translate-URL: https://translation.odoo-community.org/projects/manufacture-15-0/manufacture-15-0-mrp_production_serial_matrix/es/
Currently translated at 100.0% (42 of 42 strings) Translation: manufacture-15.0/manufacture-15.0-mrp_production_serial_matrix Translate-URL: https://translation.odoo-community.org/projects/manufacture-15-0/manufacture-15-0-mrp_production_serial_matrix/it/
DavidJForgeFlow
force-pushed
the
17.0-mig-mrp_production_serial_matrix
branch
from
cfd5a51 to
174dbde
Compare
Split the wizard into two parts to avoid a bug with many2many and onchange in a wizard.
DavidJForgeFlow
force-pushed
the
17.0-mig-mrp_production_serial_matrix
branch
from
174dbde to
37ebab8
Compare
ThiagoMForgeFlow
approved these changes
Mar 18, 2025
|
This PR has the |
|
/ocabot migration mrp_production_serial_matrix /ocabot merge nobump |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 4d4379f. Thanks a lot for contributing to OCA. ❤️ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Split the wizard into two parts to avoid a bug with many2many and onchange in a wizard.
For the rest Standard Migration to 17.0
@ForgeFlow