nus-cs2103-AY2021S2 / pe-dev-response

0 stars 0 forks source link

More ways than advertised to indicate Priority. #2270

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

image.png

log n/Household Chores d/Cleaning the kitchen p/H

This command is allowed with a priority of capital H.

Both works, just that there was little mention of the case-sensitivity of short forms.


[original: nus-cs2103-AY2021S2/pe-interim#2270] [original labels: severity.Low type.FeatureFlaw]

kieron560 commented 3 years ago

Team's Response

We have stated clearly stated in the UG that "Priorities can be shorthanded to be h , m or l respectively." and followed that with "Priorities are case insensitive," and even linked to the glossary that case insensitive means "that a word input in upper case and lower case will be taken the same way.". This should be clear to the user that even the shorthanded forms of the priority are case insensitive, and thus we do not see this as a valid issue.

Duplicate status (if any):

--