[19.0][MIG] stock_picking_analytic: Migration to 19.0 - #965
Merged
OCA-git-bot merged 29 commits intoAug 22, 2026
Conversation
…in depends + tests
- Switch field from analytic_account_id to analytic_distribution. - Assign a dummy search arg to move_ids_without_package field (stock.picking) to bypass the warning from https://github.com/odoo/odoo/blob/0f84366/odoo/fields.py#L808-L812. - Add an invisible analytic distribution field without group assignment in the picking form, due to newly added check in odoo/odoo@0501bbd.
Currently translated at 100.0% (7 of 7 strings) Translation: account-analytic-16.0/account-analytic-16.0-stock_picking_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-stock_picking_analytic/es/
Currently translated at 42.8% (3 of 7 strings) Translation: account-analytic-16.0/account-analytic-16.0-stock_picking_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-stock_picking_analytic/pt_BR/
Currently translated at 100.0% (7 of 7 strings) Translation: account-analytic-16.0/account-analytic-16.0-stock_picking_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-stock_picking_analytic/pt_BR/
Currently translated at 100.0% (7 of 7 strings) Translation: account-analytic-16.0/account-analytic-16.0-stock_picking_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-stock_picking_analytic/it/
… distribution computation Replaced the stock picking analytic distribution search and validations with a query to improve performance when there are large amounts of picking and move records. Also, updated test to handle a case where there are different analytic accounts on moves related to a picking.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-analytic-17.0/account-analytic-17.0-stock_picking_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-17-0/account-analytic-17-0-stock_picking_analytic/
Currently translated at 100.0% (6 of 6 strings) Translation: account-analytic-17.0/account-analytic-17.0-stock_picking_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-17-0/account-analytic-17-0-stock_picking_analytic/it/
Author
|
This is the same approved PR #913 |
Author
|
/ocabot merge nobump |
Contributor
|
This PR looks fantastic, let's merge it! |
Contributor
|
Congratulations, your PR was merged at b4fb9c1. Thanks a lot for contributing to OCA. ❤️ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
11 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Supersedes #913, which has been waiting on a rebase since the
stock_analyticmigration landed.This branch is #913 rebased on the current
19.0, with a single change: the last commit,[DON'T MERGE] - test-requirements.txt, is dropped. That commit only pinnedand #898 was merged on 2026-08-12, so
stock_analyticis already in19.0and the pin isobsolete. It was also the only reason the
Detect unreleased dependenciescheck was failingon #913; every other check there was green.
All 29 commits keep their original authors and author dates — nothing was squashed or
rewritten. Credit for the 19.0 migration goes to @Yusuke1998, and the module tree is
byte-identical to the one already reviewed in #913.
#913 was approved by @stferraro, @BhaveshHeliconia, @Karla-Villegas, @raulovallet and
@FrancoMaxime; cc @rousseldenis as module maintainer.
@Yusuke1998, please close #913 once this one is merged (or feel free to push the same
rebase to your branch instead and I will close this).