Skip to content

[18.0][MIG] sale_order_invoicing_picking_filter: Migration to 18.0#4147

Open
Abranes wants to merge 11 commits intoOCA:18.0from
TesseraOdoo:18.0-mig-sale_order_invoicing_picking_filter
Open

[18.0][MIG] sale_order_invoicing_picking_filter: Migration to 18.0#4147
Abranes wants to merge 11 commits intoOCA:18.0from
TesseraOdoo:18.0-mig-sale_order_invoicing_picking_filter

Conversation

@Abranes
Copy link
Copy Markdown
Member

@Abranes Abranes commented Feb 8, 2026

Standard migration to 18.0

@Abranes Abranes force-pushed the 18.0-mig-sale_order_invoicing_picking_filter branch from 3508215 to bad6fb0 Compare February 8, 2026 18:34
Copy link
Copy Markdown

@alexey-pelykh alexey-pelykh left a comment

Choose a reason for hiding this comment

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

Clean migration from 16.0 to 18.0. The key API changes have been correctly handled:

  • message_post_with_view() -> message_post_with_source() with render_values=
  • stock.move.quantity_done -> stock.move.quantity
  • product.detailed_type -> product.type (type unification in 18.0)
  • Analytic account logic simplified per 18.0 changes
  • XML views use 18.0-style invisible expressions
  • Version correctly set to 18.0.1.0.0, pyproject.toml present
  • Tests updated for 18.0 API with good coverage (7 scenarios)

CI failures are unrelated -- the failing jobs test sale_packaging_default, not this module. LGTM.

@rousseldenis
Copy link
Copy Markdown
Contributor

/ocabot migration sale_order_invoicing_picking_filter

@rousseldenis
Copy link
Copy Markdown
Contributor

@Abranes Could you rebase ?

@Abranes Abranes force-pushed the 18.0-mig-sale_order_invoicing_picking_filter branch from bad6fb0 to 8e6993c Compare April 2, 2026 09:06
Copy link
Copy Markdown
Contributor

@BhaveshHeliconia BhaveshHeliconia left a comment

Choose a reason for hiding this comment

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

Functional review LGTM!

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.

8 participants