nus-cs2103-AY2223S2 / pe-dev-response

1 stars 0 forks source link

Cannot enter amount of 2 for egg ingredient when adding a recipe #1361

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

A unit needs to be specified after every amount. This reduces the functionality as some recipes have flat amounts for ingredients (e.g 2 potatoes). Perhaps making the unit field optional would be a good improvement to the command. Screenshot 2023-04-14 150255.png


[original: nus-cs2103-AY2223S2/pe-interim#1748] [original labels: type.FunctionalityBug severity.Medium]

SeeuSim commented 1 year ago

Team's Response

We do concede that the regex/logic could be better, but typically chefs do specify: yolks, white, large, whole.

Therefore, we chose to accept this as a bug.

However, we did choose to downgrade the severity to Low. This is because:

  1. The user could provide a workaround to this which is to add a dummy unit i.e 2 whole potatoes.
  2. It only affects a small subset of ingredients, which does not affect the overall intended functionality of the application. A medium severity would indicate that this problem affects all ingredients (which is not the case).

Duplicate status (if any):

--