nus-cs2113-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Insufficient Ingredient not intuitive enough #597

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

The insufficient ingredient mesage is not intuitive for user. It could have included the amount of ingredients required for the specific number of pax as the additional info instead of just a simple statement and user will have to trial and error to figure out the max amount that can be cooked.

image.png


[original: nus-cs2113-AY1920S2/pe-interim#596]

alwayshuizhen commented 4 years ago

Team's Response

The error that the user has encountered is that there is not enough ingredients in the inventory (i.e. cooking something that needs 100*n when you only have a small amount). Specific error messages are produced when there is sufficient but contain expired ingredients. This issue could be implemented as an enhancement as this does not result in the user not being able to run the program/ missed out critical functionality and therefore not a feature flaw.

Duplicate status (if any):

--