search

nus-cs2113-AY2122S2 / pe-dev-response

0 stars 0 forks source link

Inconsistent use of plural form between "update medicines" and "view medicine" #367

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

image.png

For consistency (and to make it clear to the user that there should be multiple medicines listed), consider refactoring the code such that both use the plural (i.e. "update medicines" and "view medicines.")

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

shxr3f commented 2 years ago

Team's Response

I agree that there could have been better usage of plural and singular forms for the commands. However this is a simple issue that does not cause great displeasure to the user or make the application difficult to use. Hence I believe this of severity.Low instead of Medium.

Furthermore the functionality of each command is explained clearly in the UserGuide hence there would not be any great displeasure caused to the user.

This is similar to the bug raised #363 that calls on the use of plural and singular commands.

Duplicate status (if any):

Duplicate of #363