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

Feature Request: Price tracking for products #22

Closed
BlizzWave opened this issue Jul 22, 2018 · 3 comments
Closed

Feature Request: Price tracking for products #22

BlizzWave opened this issue Jul 22, 2018 · 3 comments

Comments

@BlizzWave
Copy link
Contributor

Adding a purchase price when purchasing products and display a graph of price over a set time period.

@berrnd
Copy link
Member

berrnd commented Jul 26, 2018

The following ideas on this:

  • To keep it simpel, no currency conversion handling, currency unit/symbol will be defined in config.php
  • Price can be entered during purchase, must be entered per purchase quantity unit, last price will be prefilled
  • For added products during inventory the last price will be used
  • Last price and small line chart of price history is shown in the product card (right side in purchase/consume/etc.)

The above is already done - maybe you can test it in the already updated English demo and give me feedback if this is what you thought of or if you have any other suggestions regarding this. (?) :)

@BlizzWave
Copy link
Contributor Author

BlizzWave commented Jul 26, 2018

COOOOL!!

I dont think there is any need to do any currency conversion as long as you can set the currency unit/ symbol in the config.
Maybe have the currency unit change depending on what language you use? (But needs a override option if you want to use something else).

This is my intial thoughts. I need to do more testing.

berrnd added a commit that referenced this issue Jul 26, 2018
…l lead to that the current price is always 0 - references #22)
@berrnd
Copy link
Member

berrnd commented Aug 4, 2018

I think I'll close this for now, the base tracking should work - better/more statistics still needed everywhere...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants