pos icon indicating copy to clipboard operation
pos copied to clipboard

[16.0][MIG] pos_require_product_quantity: Migration to 16.0

Open mihien opened this issue 3 months ago • 4 comments

Internal task T14562

mihien avatar Sep 27 '25 22:09 mihien

@mihien failed test on runboat : there are pos lines with quantity zero but no warning is shown

@polchampion You first have to check the option "Require Product Quantity" in the pos settings to activate it. The option was already in the settings in 12.0 but wasn't used in the module's logic. Whether you checked it or not, quantities of 0 used to result in a popup either way. If that's better, I could have it turned on by default when installing the module or if we want the same behaviour as in 12.0, I can remove the option from the settings entirely

mihien avatar Oct 08 '25 23:10 mihien

@mihien sucessfully tested, can you add a "configuration" section in the readme so that it's clear how it needs to be set up ?

also, why put this in the session properties and not in the general pos settings? it seems not to respect the general logic

@polchampion I've set the setting to true by default, so that there is no need for a configuration anymore and took the opportunity to move it to the general pos settings as well.

mihien avatar Oct 24 '25 08:10 mihien

I've set the setting to true by default, so that there is no need for a configuration anymore and took the opportunity to move it to the general pos settings as well.

perfect thank you!

polchampion avatar Oct 24 '25 12:10 polchampion

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). 🤖

OCA-git-bot avatar Nov 12 '25 11:11 OCA-git-bot