search

Ruizhi2001 / pe

0 stars 0 forks source link

Inaccurate ingredient names #4

Open Ruizhi2001 opened 7 months ago

Ruizhi2001 commented 7 months ago

I added the ingredient potato but the displayed ingredient is POTATO. It is not stated anywhere that the ingredients would be stored as upper case.

image.png

image.png

nus-pe-script commented 7 months ago

Team's Response

It is by design to store as upper case for the following reasons:

  1. This is to standardise the overall display aesthetics of the product
  2. This is to help users navigate the product. As parameters are case-insensitive, storing ingredients as the exact case that they were stored in will further inconvenience the user with this feature

It should also be noted that this should not inconvenience the average user. As such, we don't think that this is bug

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: 1. As the parameters are case insensitive, there is no need to capitalise the ingredients anyways, so the point on aesthetics seems to be irrelevant here.

  1. The search being case insensitive, can still be implemented without having to capitalise everything.
    It inconveniences the user when the user wants to quickly glance at his ingredients but realises that all the ingredients are in the uppercase version of what he has typed. He then has to recognise what the ingredient name in uppercase is rather than referring back to how he had typed it in.