-
-
Notifications
You must be signed in to change notification settings - Fork 154
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 multiple named wishlists per user #2147
Comments
Thinking about this, the default list is actually "the wishlist", the others are "shopping lists", as in products which are in some way special for the user to be grouped together. For example, the user could have a shopping list called "Cleaning supplies" prepared, they occasionally put the contents of that list in the cart and buy it, they still want to keep the list after they're done. They don't "wish" it, they want to have a list prepared so they don't need to always look for items they always get. This is a subtle wording difference, functionally it's exactly the same. |
Is this done after #2150 has been merged? |
@BlackbitDevs no, this is different: that one allows you to make a whishlist public. This issue is describing being able to have multiple wishlists. |
We could add support to allow multiple wishlists per user. This would mean you could manage your own wish lists and add a new one or delete old ones.
Things to consider:
name
, it would allow to auto-create the named wishlist if there's none of that name alreadyname
argument in 1)Example:
Default
is created and marked as "current" (default name up for discussion)Default
wishlistKitchen
, a new wishlistKitchen
is created and marked as "current"Kitchen
wishlistKitchen
wishlist is displayed (current)Default
,Default
wishlist is displayed and marked as "current"Default
wishlist, it's deleted and summary showsKitchen
wishlist (since it was "current", we now use last modified)Kitchen
wishlist, it's deleted and summary shows "No items in wishlist"Default
is created and marked as "current" (same as 2)The text was updated successfully, but these errors were encountered: