-
-
Notifications
You must be signed in to change notification settings - Fork 724
[18.0][MIG] stock_picking_import_serial_number: Migration to version 18.0 #2133
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
base: 18.0
Are you sure you want to change the base?
[18.0][MIG] stock_picking_import_serial_number: Migration to version 18.0 #2133
Conversation
[UPD] Update stock_picking_import_serial_number.pot
You can't use a static assignation in kw args, and less the empty string, which is assigned as is on the char fields (and being different from NULL values).
Don't pass `default_code` in vals if no reference is provided, for letting other modules to work properly in integration tests like `product_code_mandatory` + `product_sequence`. stock_picking_import_serial_number 13.0.1.0.2
[UPD] Update stock_picking_import_serial_number.pot [BOT] post-merge updates
Currently translated at 100.0% (29 of 29 strings) Translation: stock-logistics-workflow-15.0/stock-logistics-workflow-15.0-stock_picking_import_serial_number Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-15-0/stock-logistics-workflow-15-0-stock_picking_import_serial_number/es/
Translated using Weblate (Italian) Currently translated at 100.0% (29 of 29 strings) Translation: stock-logistics-workflow-15.0/stock-logistics-workflow-15.0-stock_picking_import_serial_number Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-15-0/stock-logistics-workflow-15-0-stock_picking_import_serial_number/it/ Translated using Weblate (Italian) Currently translated at 100.0% (29 of 29 strings) Translation: stock-logistics-workflow-15.0/stock-logistics-workflow-15.0-stock_picking_import_serial_number Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-15-0/stock-logistics-workflow-15-0-stock_picking_import_serial_number/it/
Currently translated at 100.0% (29 of 29 strings) Translation: stock-logistics-workflow-15.0/stock-logistics-workflow-15.0-stock_picking_import_serial_number Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-15-0/stock-logistics-workflow-15-0-stock_picking_import_serial_number/it/
Adds the possibility to set the package name (stock.quant.package) in the import file to be auto-created with that name. TT47271 [UPD] Update stock_picking_import_serial_number.pot [BOT] post-merge updates Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-workflow-15.0/stock-logistics-workflow-15.0-stock_picking_import_serial_number Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-15-0/stock-logistics-workflow-15-0-stock_picking_import_serial_number/
Currently translated at 100.0% (32 of 32 strings) Translation: stock-logistics-workflow-15.0/stock-logistics-workflow-15.0-stock_picking_import_serial_number Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-15-0/stock-logistics-workflow-15-0-stock_picking_import_serial_number/it/
Currently translated at 100.0% (32 of 32 strings) Translation: stock-logistics-workflow-16.0/stock-logistics-workflow-16.0-stock_picking_import_serial_number Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-16-0/stock-logistics-workflow-16-0-stock_picking_import_serial_number/it/
d10056a
to
bd092ee
Compare
bd092ee
to
06d274a
Compare
/ocabot migration stock_picking_import_serial_number |
cls.lot_obj = cls.env["stock.lot"] | ||
cls.warehouse = cls.env.ref("stock.warehouse0") | ||
cls.picking_type_in = cls.env.ref("stock.picking_type_in") | ||
cls.picking_type_in = cls.env.ref("stock.picking_type_out") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why changing this? We want to try on incoming receptions.
self.assertIn("LOT-3", lot_names) | ||
self.assertFalse(smls[0].result_package_id) | ||
|
||
@mute_logger("odoo.models.unlink") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't remove these tests, but adapt them. We also want to make this work on outgoing pickings.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed those tests as I understood that they are checking show_reserved
and move_line_nosuggest_ids
, that are no longer in 18. So if I adapt them I will be checking for the same thing as test above. Am I wrong?
name="%(action_import_serial_number)d" | ||
string="Import S/N" | ||
attrs="{'invisible': ['|', ('state', '!=', 'assigned'), ('picking_type_code', '!=', 'incoming')]}" | ||
invisible="state != 'assigned' or picking_type_code != 'incoming'" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pedrobaeza it should only work at the moment whit incoming, right? Anyways i do not see why it wont work with outgoings.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With outgoing pickings, you have to check also for reservations, so there are so few steps. That's why the extra tests you removed.
cc @Tecnativa TT57280
ping @victoralmau @sergio-teruel
I change the test as show_reserved is no longer available in 18.0, I did remove that part as IMHO I think it is unnecessary.