Skip to content

Conversation

@ccarrenoc
Copy link

Changes

  • Migrated module pos_operating_unit from 17.0 to 18.0.
  • Updated __manifest__.py version to 18.0.1.0.0.
  • Adjusted imports and dependencies for Odoo 18.0.
  • Applied OCA code quality hooks (pre-commit).
  • Updated CONTRIBUTORS.md with Nybble Group and Cristiam Carreño.

Notes

  • No functional changes beyond the migration.
  • Migration scripts not required (migrations/18.0.1.0.0/) as no model/field renames were detected.
  • CLA already signed.

--
cc @OCA/maintainers

@AaronHForgeFlow
Copy link
Contributor

Hi! Thanks for the migration. The commit history is missing. For reference, this is the OCA documentation for migrations: https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-18.0

@ccarrenoc
Copy link
Author

@AaronHForgeFlow Thanks for the feedback! I’ve updated the branch using oca-port so that the full commit history of pos_operating_unit is now preserved. Please let me know if there’s anything else I should adjust.

@AaronHForgeFlow
Copy link
Contributor

Thank you but it seems there is something not correct. The first commit should not be this:

image

It is strange you have merge conflicts, but in any case there should be be resolved without extra commits. Just please be sure you have your OCA remote updated before you start preparing the branch for the migration.

ilyasProgrammer and others added 15 commits September 2, 2025 14:02
Currently translated at 100.0% (11 of 11 strings)

Translation: operating-unit-14.0/operating-unit-14.0-pos_operating_unit
Translate-URL: https://translation.odoo-community.org/projects/operating-unit-14-0/operating-unit-14-0-pos_operating_unit/it/
Currently translated at 100.0% (11 of 11 strings)

Translation: operating-unit-14.0/operating-unit-14.0-pos_operating_unit
Translate-URL: https://translation.odoo-community.org/projects/operating-unit-14-0/operating-unit-14-0-pos_operating_unit/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: operating-unit-17.0/operating-unit-17.0-pos_operating_unit
Translate-URL: https://translation.odoo-community.org/projects/operating-unit-17-0/operating-unit-17-0-pos_operating_unit/
Currently translated at 100.0% (9 of 9 strings)

Translation: operating-unit-17.0/operating-unit-17.0-pos_operating_unit
Translate-URL: https://translation.odoo-community.org/projects/operating-unit-17-0/operating-unit-17-0-pos_operating_unit/it/
@ccarrenoc ccarrenoc force-pushed the 18.0-mig-pos_operating_unit branch from 1c7b956 to f50e37b Compare September 2, 2025 19:34
@ccarrenoc ccarrenoc force-pushed the 18.0-mig-pos_operating_unit branch from a66a885 to 5553c6f Compare September 2, 2025 20:59
@ccarrenoc
Copy link
Author

ccarrenoc commented Sep 2, 2025

All checks are now green, and the commit history has been preserved with oca-port. The branch has been cleaned to a single migration commit. This PR is ready for final review.

@AaronHForgeFlow
Copy link
Contributor

It looks good now. Thank you!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants