Skip to content

[MIG] pos_lot_barcode: Migration to 19.0#1465

Open
mkoeck wants to merge 32 commits intoOCA:19.0from
mkoeck:19.0-mig-pos_lot_barcode
Open

[MIG] pos_lot_barcode: Migration to 19.0#1465
mkoeck wants to merge 32 commits intoOCA:19.0from
mkoeck:19.0-mig-pos_lot_barcode

Conversation

@mkoeck
Copy link
Copy Markdown

@mkoeck mkoeck commented Dec 8, 2025

No description provided.

grindtildeath and others added 30 commits December 8, 2025 21:47
Currently translated at 100.0% (9 of 9 strings)

Translation: pos-16.0/pos-16.0-pos_lot_barcode
Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_lot_barcode/es/
Currently translated at 100.0% (9 of 9 strings)

Translation: pos-16.0/pos-16.0-pos_lot_barcode
Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_lot_barcode/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: pos-17.0/pos-17.0-pos_lot_barcode
Translate-URL: https://translation.odoo-community.org/projects/pos-17-0/pos-17-0-pos_lot_barcode/
Currently translated at 100.0% (6 of 6 strings)

Translation: pos-17.0/pos-17.0-pos_lot_barcode
Translate-URL: https://translation.odoo-community.org/projects/pos-17-0/pos-17-0-pos_lot_barcode/fr/
Currently translated at 100.0% (6 of 6 strings)

Translation: pos-17.0/pos-17.0-pos_lot_barcode
Translate-URL: https://translation.odoo-community.org/projects/pos-17-0/pos-17-0-pos_lot_barcode/it/
Currently translated at 100.0% (6 of 6 strings)

Translation: pos-17.0/pos-17.0-pos_lot_barcode
Translate-URL: https://translation.odoo-community.org/projects/pos-17-0/pos-17-0-pos_lot_barcode/es/
Currently translated at 100.0% (6 of 6 strings)

Translation: pos-17.0/pos-17.0-pos_lot_barcode
Translate-URL: https://translation.odoo-community.org/projects/pos-17-0/pos-17-0-pos_lot_barcode/fr/
@mkoeck
Copy link
Copy Markdown
Author

mkoeck commented Dec 8, 2025

I also still have an open PR here #1430 for the same module. Since this is a migration PR I did not include the changes from #1430 but the issue still persists. Would be nice to get some feedback on both.

@mkoeck mkoeck mentioned this pull request Dec 8, 2025
7 tasks
@mkoeck mkoeck marked this pull request as draft December 8, 2025 22:23
@mkoeck mkoeck marked this pull request as ready for review December 9, 2025 09:24
@mkoeck
Copy link
Copy Markdown
Author

mkoeck commented Dec 9, 2025

Fixed all the existing tests which now pass and added a test for the issue described in #1430 which now fails. Please advise on how to proceed with this.

Copy link
Copy Markdown
Contributor

@ivantodorovich ivantodorovich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Can you clean up your migration commits? pehaps squash all or most of them?

/ocabot migration pos_lot_barcode

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Apr 9, 2026
@mkoeck mkoeck force-pushed the 19.0-mig-pos_lot_barcode branch from d7a5268 to f984a6f Compare April 10, 2026 07:34
@mkoeck
Copy link
Copy Markdown
Author

mkoeck commented Apr 10, 2026

@ivantodorovich squashed the commits into two separate parts: 1. the normal migration, 2. the fix of the issue described in #1430

I hope its fine to include that one here as well.

The CI pipeline seems to fail, even though the tests are passing. I am not sure what is going on here. Seems to be a false positive due to the Chrome shutdown warning?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.