nus-cs2113-AY2122S2 / pe-dev-response

0 stars 0 forks source link

Commands for viewing doctors, patients, medicines should probably be pluralized #363

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

image.png

The singular "doctor" in "view doctor" seems to invite the user to view a singular doctor instance, and it would therefore seem intuitive to write "view doctor /info [NRIC]." However, this command allows the user to view all doctors, and so it would likely be less confusing if the command was "view doctors" instead.


[original: nus-cs2113-AY2122S2/pe-interim#408] [original labels: type.FeatureFlaw severity.Medium]

shxr3f commented 2 years ago

Team's Response

Yes I agree that a better usage of plural and singular forms might be more intuitive. However the functionalities of these commands have been clearly explained in the UserGuide. I do not think this is an issue of Medium severity, there will be some occasions where it might hinder the user experience but however after reading the User Guide and having used the application this should not give any issues to the user. Hence I believe this is an issue of lower severity.

Duplicate status (if any):

--