search

nus-cs2113-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Delete food bank functionality does not perform as expected when the input is in uppercase. #2962

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

The UG mentioned that the commands are not case-sensitive. However, the delete food bank functionality does not perform as expected when the input is in uppercase.

image.png

[original: nus-cs2113-AY2122S1/pe-interim#2954] [original labels: severity.High type.FunctionalityBug]

tlyi commented 2 years ago

Team's Response

We think that this is a documentation bug and not a functionality bug as this is how we intended the app to work. In the UG, when we said that commands are case insensitive, we were actually referring to the command keyword like add, delete, view, etc. Prefixes like f/, e/, h/, w/ still needs to be in lowercase. Therefore, in this case, DELETE fbank/all would have worked. However, we acknowledge that this was not clearly stated in the UG and could have caused confusion to readers.

Duplicate status (if any):

--