Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
nus-se-bot opened this issue Nov 12, 2021 · 1 comment

Comments

@nus-se-bot
Copy link

nus-se-bot commented Nov 12, 2021

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
Copy link
Contributor

tlyi commented Nov 15, 2021

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):

--

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants