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

NEW : allow stock management by product #29056

Open
wants to merge 21 commits into
base: develop
Choose a base branch
from

Conversation

thomas-Ngr
Copy link
Contributor

NEW : allow stock management by product

When using product stocks, users may want to have some products for which they don't want to manage stocks, for example in case of subcontracting.

This PR:

  • adds a field "Stockable product" on product card (editable on update, not on create)
  • sets this option to 1 by default
  • hides the "stocks" tabs if this option is not set
  • does not manage stock movements for a product if this option is not set.

Discussion started on #26184 and #26200

SQL Part : #26190

/**
* Advanced feature: assembly kit
*/
const TYPE_ASSEMBLYKIT = 2;
Copy link
Member

Choose a reason for hiding this comment

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

It seems this is not related to the feature stockable or not.
Thanks to split this other feature into a different PR.

@eldy eldy added the PR to split into several PRs PR is too large or not "atomic" on feature. Please try to push it in several independent PR. label Mar 25, 2024
@thomas-Ngr
Copy link
Contributor Author

@eldy I have removed the only faulty line, can you have a look ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR to split into several PRs PR is too large or not "atomic" on feature. Please try to push it in several independent PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants