Skip to content

Conversation

@jbaudoux
Copy link
Contributor

@jbaudoux jbaudoux commented Jun 6, 2025

cc @mt-software-de @rousseldenis

Depends on:

Includes:

  • FIX when there is a need on a child location of the orderpoint location. The product was not considered.

@jbaudoux
Copy link
Contributor Author

jbaudoux commented Jun 6, 2025

/ocabot migration stock_location_orderpoint

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Jun 6, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Jun 6, 2025
7 tasks
@jbaudoux jbaudoux force-pushed the 18.0-mig-stock_location_orderpoint branch 2 times, most recently from 607a55d to c2ff5a5 Compare June 6, 2025 15:46
@jbaudoux jbaudoux marked this pull request as ready for review June 6, 2025 15:51
@jbaudoux jbaudoux force-pushed the 18.0-mig-stock_location_orderpoint branch 3 times, most recently from a0c6abc to ec028a6 Compare June 6, 2025 22:31
@jbaudoux
Copy link
Contributor Author

jbaudoux commented Jun 7, 2025

This test is sometimes failing:
image

@jbaudoux jbaudoux force-pushed the 18.0-mig-stock_location_orderpoint branch 2 times, most recently from bdfd4f7 to 4b429f0 Compare June 7, 2025 06:31
@mt-software-de
Copy link
Contributor

This test is sometimes failing: image

Why sometimes? :-)

Copy link
Contributor

@mt-software-de mt-software-de left a comment

Choose a reason for hiding this comment

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

Thx. This is fine also your FIX.
Just the test-requirements.txt commit needs to be removed.

Comment on lines 1 to 2
odoo-addon-stock_available_base_exclude_location @ git+https://github.com/OCA/stock-logistics-availability.git@refs/pull/51/head#subdirectory=stock_available_base_exclude_location
odoo_test_helper
Copy link
Contributor

Choose a reason for hiding this comment

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

Those changes can be dropped. Since the PR is merged.

@jbaudoux
Copy link
Contributor Author

Why sometimes? :-)

@mt-software It failed locally twice and then never again but I changed nothing...

mymage and others added 22 commits June 11, 2025 11:22
Currently translated at 100.0% (49 of 49 strings)

Translation: stock-logistics-orderpoint-16.0/stock-logistics-orderpoint-16.0-stock_location_orderpoint
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-orderpoint-16-0/stock-logistics-orderpoint-16-0-stock_location_orderpoint/it/
As in some configuration, Reception could be under the refilled location
(e.g. : Stock) and if an incoming move is confirmed to the refilled location,
no replenishment will be triggered as the virtual_available quantity will
be > 0.
…ntities_dict with recordset"

This reverts commit 0e0e0d0.
Uses the last cron run date as date criteria into the domain used to detect new incming moves that could trigger a replenishment when the replenishments scheduler is triggered by cron.
Uses the move date in place of the write_date to filter moves. write_date is not indexed and we are only interested when moves are created or are done.
Split the query to get moves that could trigger a replenishment: One for the incoming and an other one for the consuming. This is required to avoid a query with a OR between 2 complex domain. We prefer to make 2 queries instead of 1 with a big OR statement because the query planner is not able to use the indexes properly
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: stock-logistics-orderpoint-16.0/stock-logistics-orderpoint-16.0-stock_location_orderpoint
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-orderpoint-16-0/stock-logistics-orderpoint-16-0-stock_location_orderpoint/
Currently translated at 100.0% (55 of 55 strings)

Translation: stock-logistics-orderpoint-16.0/stock-logistics-orderpoint-16.0-stock_location_orderpoint
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-orderpoint-16-0/stock-logistics-orderpoint-16-0-stock_location_orderpoint/it/
Currently translated at 70.9% (39 of 55 strings)

Translation: stock-logistics-orderpoint-16.0/stock-logistics-orderpoint-16.0-stock_location_orderpoint
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-orderpoint-16-0/stock-logistics-orderpoint-16-0-stock_location_orderpoint/fr/
Currently translated at 100.0% (55 of 55 strings)

Translation: stock-logistics-orderpoint-16.0/stock-logistics-orderpoint-16.0-stock_location_orderpoint
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-orderpoint-16-0/stock-logistics-orderpoint-16-0-stock_location_orderpoint/tr/
Currently translated at 100.0% (55 of 55 strings)

Translation: stock-logistics-orderpoint-16.0/stock-logistics-orderpoint-16.0-stock_location_orderpoint
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-orderpoint-16-0/stock-logistics-orderpoint-16-0-stock_location_orderpoint/tr/
Compute orderpoint for products when the need is on a child location of the orderpoint location
@jbaudoux jbaudoux force-pushed the 18.0-mig-stock_location_orderpoint branch from a39e326 to d904273 Compare June 11, 2025 09:23
@jbaudoux
Copy link
Contributor Author

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-38-by-jbaudoux-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 220b4ac into OCA:18.0 Jun 11, 2025
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at f45a0b1. 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.

9 participants