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

Add inventory UI to manage inventory warning threshold per option #4358

Closed
3 tasks done
willopez opened this issue Jun 22, 2018 · 5 comments
Closed
3 tasks done

Add inventory UI to manage inventory warning threshold per option #4358

willopez opened this issue Jun 22, 2018 · 5 comments
Assignees
Labels
bug For issues that describe a defect or regression in the released software
Milestone

Comments

@willopez
Copy link
Member

Prerequisites

  • Are you running the latest version?
  • Are you able to consistently reproduce the issue?
  • Did you search the issue queue for existing issue? Search issues

Issue Description

Currently there is no way to set the "Warn at" threshold level per option, it can only be set at the variant level. This is incorrect, since we are tracking inventory per option and per variant. This causes a problem when determining if a "limited supply" badge should be displayed for an option. Without the "Warn at", level set per option, it's not possible to make that determination. This issue will address this need by adding a "Warn at" input field at the option level, refer to image below for a visual. A background discussion is found here: #4342

warn_at

Possible Solution

Adding UI to set "Warn at" level per option.

@willopez willopez added this to the Humbolt milestone Jun 22, 2018
@willopez willopez added the bug For issues that describe a defect or regression in the released software label Jun 22, 2018
@spencern
Copy link
Contributor

Are we currently using the warn at level from the variant?
Can we imply that the option warn at levels are the same as the level set for variants for now?

@willopez
Copy link
Member Author

@spencern Yes, the parent top level variant's warn at level could be used as a work around. Nat suggested that here: #4342 (review). I am not fond of that solution, cause it will lead to confusion in the future, and it's not correct, given that inventory is tracked at the variant/option level.

@spencern spencern added the UIX label Jun 25, 2018
@spencern
Copy link
Contributor

@willopez I agree it's not a great solution in the long run. If it's a trivial thing to add to the product option operator panel, let's schedule it for our next sprint.

@willopez
Copy link
Member Author

@spencern I looked at what it would take, and it's like a 5-8 effort.

@spencern
Copy link
Contributor

Thanks @willopez. Not quite trivial, let's hold off on this for now and plan to solve it when we revisit product management.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug For issues that describe a defect or regression in the released software
Projects
None yet
Development

No branches or pull requests

4 participants