operating-unit icon indicating copy to clipboard operation
operating-unit copied to clipboard

[18.0][MIG] pos_operating_unit: migrate to 18.0

Open ccarrenoc opened this issue 4 months ago • 5 comments

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

ccarrenoc avatar Sep 01 '25 14:09 ccarrenoc

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

AaronHForgeFlow avatar Sep 01 '25 15:09 AaronHForgeFlow

@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.

ccarrenoc avatar Sep 01 '25 17:09 ccarrenoc

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.

AaronHForgeFlow avatar Sep 02 '25 13:09 AaronHForgeFlow

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.

ccarrenoc avatar Sep 02 '25 21:09 ccarrenoc

It looks good now. Thank you!

AaronHForgeFlow avatar Sep 03 '25 09:09 AaronHForgeFlow