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 Feature: Shopping List - show unit price with PriceQU #743

Closed
alkuzman opened this issue Aug 2, 2023 · 2 comments · Fixed by #809
Closed

New Feature: Shopping List - show unit price with PriceQU #743

alkuzman opened this issue Aug 2, 2023 · 2 comments · Fixed by #809
Assignees
Labels
enhancement New feature or request

Comments

@alkuzman
Copy link

alkuzman commented Aug 2, 2023

If you use Grocy for managing recipes, then probably you too store most of the products in grams and milliliters. This results in the unit price in the shopping list = 0 for almost all products. Since Grocy 4.0.0 we can use the newly provided PriceQU for showing this price which will make it more relevant in most markets that provide prices in kilogram and liter.

@dominiczedler
Copy link
Collaborator

dominiczedler commented Aug 7, 2023

Thanks for the suggestion, can you please describe again what is the current state in our app for this and what you expect with an improvement? (I didn't understand it enough, sorry)

@dominiczedler dominiczedler self-assigned this Aug 7, 2023
@alkuzman
Copy link
Author

alkuzman commented Aug 7, 2023

I made the same request in the Grocy server repo. It was just released. grocy/grocy#2294. There I maybe explained it a bit better.

I believe the solution there was to use the QU specified in the shopping list entry. At least that was what I inferred from the commit.

@dominiczedler dominiczedler added the enhancement New feature or request label Aug 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants