search

zhengster / pe

0 stars 0 forks source link

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

Open zhengster opened 2 years ago

zhengster 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.

soc-se-bot commented 2 years ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

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.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your reason]