Skip to content

Conversation

@apami0111
Copy link
Contributor

  • Updated module version from 18.0.1.0.0 to 19.0.1.0.0.
  • Adapted _get_new_picking_values method in models/stock_move.py:
    replaced group_id.sale_id.priority with reference_ids.sale_ids.priority
    according to new relational model in Odoo 19.
  • Updated test class inheritance in tests/test_sale_stock.py:
    replaced TestSaleCommonBase with TestSaleCommon.
  • Replaced deprecated method setup_sale_configuration_for_company
    with collect_company_accounting_data.
  • Updated sale order line field from product_uom to product_uom_id
    following Odoo 19 conventions.

SimoRubi and others added 16 commits October 28, 2025 19:23
* [ADD] sale_order_priority

* Code improvements

* [FIX] Better not to call write in inverse
Currently translated at 100.0% (8 of 8 strings)

Translation: sale-workflow-14.0/sale-workflow-14.0-sale_order_priority
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-14-0/sale-workflow-14-0-sale_order_priority/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: sale-workflow-15.0/sale-workflow-15.0-sale_order_priority
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-15-0/sale-workflow-15-0-sale_order_priority/
Currently translated at 100.0% (7 of 7 strings)

Translation: sale-workflow-17.0/sale-workflow-17.0-sale_order_priority
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-17-0/sale-workflow-17-0-sale_order_priority/it/
Copy link
Contributor

@celm1990 celm1990 left a comment

Choose a reason for hiding this comment

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

LGTM
@rousseldenis @marcelsavegnago @BhaveshHeliconia @pedrobaeza could you please review this?

@pedrobaeza
Copy link
Member

/ocabot migration sale_order_priority

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Nov 21, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 21, 2025
97 tasks
Copy link
Contributor

@Reyes4711-S73 Reyes4711-S73 left a comment

Choose a reason for hiding this comment

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

LGTM

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@celm1990
Copy link
Contributor

ping @pedrobaeza

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 19.0-ocabot-merge-pr-3999-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 3334b59 into OCA:19.0 Jan 16, 2026
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at f66a06b. Thanks a lot for contributing to OCA. ❤️

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.