[18.0][MIG] stock_mts_mto_rule: Migration to 18.0#2489
Open
[18.0][MIG] stock_mts_mto_rule: Migration to 18.0#2489
Conversation
|
I've been testing the module and it seems to be working fine when combined with purchases and manufacturing, stock movements makes sense, from what I can see |
arielbarreiros96
suggested changes
Jan 5, 2026
ypapouin
force-pushed
the
18.0-mig-stock_mts_mto_rule
branch
2 times, most recently
from
2fc0a46 to
82c1844
Compare
ypapouin
force-pushed
the
18.0-mig-stock_mts_mto_rule
branch
from
82c1844 to
2caddab
Compare
… procurement rules to handle the mts+mto scenario
Add explanation for finding this setting
Conforms to the latest README template: bugtracker, runbot etc. Fixes bugtracker URL on some modules. States OCA as maintainer, removes other contributors from the 'Maintainer' section.
…en warehouse is two/three steps delivery propagate move by mts-mto in pick/pack/out
OCA Transbot updated translations from Transifex
…least another procurement
There's a traceback when renaming warehouse code (without renaming warehouse name), as name argument is False.
OCA Transbot updated translations from Transifex OCA Transbot updated translations from Transifex
This method is called from the write() method the one how support multiple records an api.multi method. I was making some test when I tried to activate the mto+mts option for multiple warehouse and the next error appears: ``ValueError: Expected singleton`` In order to fix this error I only added a loop to manage the multiple registers.
OCA Transbot updated translations from Transifex
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-warehouse-13.0/stock-logistics-warehouse-13.0-stock_mts_mto_rule Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-13-0/stock-logistics-warehouse-13-0-stock_mts_mto_rule/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_mts_mto_rule Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_mts_mto_rule/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_mts_mto_rule Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_mts_mto_rule/
Currently translated at 100.0% (13 of 13 strings) Translation: stock-logistics-warehouse-17.0/stock-logistics-warehouse-17.0-stock_mts_mto_rule Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-17-0/stock-logistics-warehouse-17-0-stock_mts_mto_rule/it/
Currently translated at 100.0% (13 of 13 strings) Translation: stock-logistics-warehouse-17.0/stock-logistics-warehouse-17.0-stock_mts_mto_rule Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-17-0/stock-logistics-warehouse-17-0-stock_mts_mto_rule/es/
Currently translated at 100.0% (13 of 13 strings) Translation: stock-logistics-warehouse-17.0/stock-logistics-warehouse-17.0-stock_mts_mto_rule Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-17-0/stock-logistics-warehouse-17-0-stock_mts_mto_rule/it/
ypapouin
force-pushed
the
18.0-mig-stock_mts_mto_rule
branch
from
2caddab to
b8aa668
Compare
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.
New PR written from scratch since #2425 made the tests work with the code instead of the other way around, there were too many things to re-fix.
location_dest_from_rule=Trueto ensure that stock move'slocation_dest_idis properly computed.Note that I made a port of #2098 + #2331 that could be added in this PR if necessary (waiting for a feedback from original author)
Another thing that made me scratch my head is the introduction of the
location_final_idon thestock.movemodel that I though was a way to know the final product location when there were chained moves, but that's not the case since openupgrade just renamelocation_dest_idtolocation_final_idand let thelocation_dest_idbe auto-computed.