search

nus-tic4002-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Valid commands typed with Capital letter / sentence case is parsed and said to be invalid #108

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Command Used: Clear List Help

These are commands that do not take in additional parameters as well.

This could also be considered up for a very low Severity functionality / documentation bug.

Expected Result: Commands are valid. They are not parsed and considered valid despite this although there is caveat already mentioned in the UG stating Upper case words are input parameters, this is sentence case.. Just saying.

Observed Result: Command is considered invalid / unknown.

Scenario: If not clearly stated in the User Guide, a user could be potentially confused by the sentence case commands / inputs not being considered valid.

image.png

image.png

image.png

[original: nus-tic4002-AY2021S2/pe-interim#108] [original labels: severity.VeryLow type.FunctionalityBug]

binbinhui commented 3 years ago

Team's Response

image.png

Base on the user guide clearly mentions the format of the command line which in help, list, clear. I agree that we can improve by using case insensitive.

Duplicate status (if any):

--