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

Allow back in stock subscriptions also for ManageStockByAttributes inventory method #4955

Closed
GidanMX2 opened this issue Jul 31, 2020 · 2 comments

Comments

@GidanMX2
Copy link

GidanMX2 commented Jul 31, 2020

nopCommerce version: 4.30

Steps to reproduce the problem:

If you have a product with inventory method set on "ManageStockByAttributes" there isn't the option to allow back in stock subscriptions. Reading the code I think that this is due to how attribute selection is handled by nopCommerce, now the selection of the attribute is handled client side by js code causing a whole series of problems (es. missing of the ability to link to a specific product/attribute selection, and so the very first price rendered by the server is always that of preselected attribute) while in my opinion it would be better capturing the change event of the select (for attributes with drop down list selection) and then make an ajax call to the same method of ProductController (ProductDetails method) passing an extra argument that specify which attributes are selected, then it could be possible to edit the PrepareProductDetailsModel to support AllowBackInStockSubscriptions even with ManageStockByAttributes inventory type.
Someone is already working on this thing, is it a change that could be of interest?

@ipetk0v
Copy link

ipetk0v commented Feb 8, 2022

Many of our customers want this functionality. It makes sense that when we have a clothing store and a certain size is over, I want to subscribe to a specific size, not the whole product. I hope this will be introduced in a new version soon.

@AndreiMaz AndreiMaz removed this from the Version 4.70 milestone Nov 16, 2022
@RomanovM
Copy link
Contributor

After some consideration we've decided not to implement this functionality out of the box and leave it for customization.

@RomanovM RomanovM closed this as not planned Won't fix, can't repro, duplicate, stale Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants