karlomikus / vue-salt-rim

Salt Rim is a Vue based web client used for connecting to Bar Assistant server instance.
https://barassistant.app
MIT License
93 stars 17 forks source link

[Bug] Variable Amount feature for cocktail ingredients has units issue #120

Closed zhdenny closed 11 months ago

zhdenny commented 11 months ago

Found a bug on V3.

To reproduce:

  1. Edit/Add a cocktail
  2. Add an ingredient to cocktail
  3. Checkmark "Has Variable Amount"
  4. Use oz (ounces) and fill in the "Amount" and "Amount Max" fields
  5. Save
  6. Observe the Amount Max is shown incorrectly.

You will notice that the "Amount Max" field is forced into ml units. So for example, if you put 2 ounces as the "Amount Max".....BarAssistant will read that as 2 ml instead of ounces. So when you view the recipe in ounces, it will show "0.06666666666666667 oz"