Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][ADD] internal stock quant package #34

Open
wants to merge 2 commits into
base: 16.0
Choose a base branch
from

Conversation

glitchov
Copy link

@glitchov glitchov commented Feb 9, 2023

Defines internal package used in picking process and are emptied (or not upon config) at 'put in pack' or when the picking is validated.

@glitchov glitchov force-pushed the 16.0-add-internal_stock_quant_package-hda branch 2 times, most recently from bf651b6 to 1d23084 Compare February 9, 2023 13:09
@glitchov glitchov force-pushed the 16.0-add-internal_stock_quant_package-hda branch 2 times, most recently from acc0272 to 776a354 Compare February 10, 2023 08:20
internal_stock_quant_package/readme/USAGE.rst Outdated Show resolved Hide resolved
requirements-test.txt Outdated Show resolved Hide resolved
@glitchov glitchov force-pushed the 16.0-add-internal_stock_quant_package-hda branch from 776a354 to 472c75e Compare February 10, 2023 12:52
Copy link

@lmignon lmignon left a comment

Choose a reason for hiding this comment

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

LGTM (Code review + functional tests)

@rousseldenis rousseldenis added this to the 16.0 milestone May 19, 2023
@rousseldenis rousseldenis force-pushed the 16.0-add-internal_stock_quant_package-hda branch from 5450d64 to 46777fb Compare May 19, 2023 14:03
glitchov and others added 2 commits May 3, 2024 09:01
As Odoo unpack() is calling the _merge_quants() function which can be bulky,
don't call it if not necessary
@lmignon lmignon force-pushed the 16.0-add-internal_stock_quant_package-hda branch from 46777fb to 53151c2 Compare May 3, 2024 07:01
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.

5 participants