search

ibrahimisramos / pe

0 stars 0 forks source link

Unclear of what is allowed or not (for boundary values), making it not so user friendly #13

Open ibrahimisramos opened 2 years ago

ibrahimisramos commented 2 years ago

There are no sample values for things such as side effects, and if I am a new user to the programme, I would be scratching my head. Also, if there are no side effects, there is little detail on what I should or can do.

image.png

This applies to adding a new doctor. What I am a doctor without a specialisation?

image.png Such cases could have been mentioned slightly to aid a poor newbie user

soc-se-bot commented 2 years ago

Team's Response

For the first issue pointed out, as a target user (which is a clinic administrator), they can refer to the actual medicine to see the side effects of the medicine. This application is meant to support the work of the clinic administrator only, hence the clinic administrator can get certain details accurately elsewhere instead of relying on this application for potentially inaccurate example values.

For the second issue, the feature is meant to help clinic administrators identify the specialization that doctors potentially have, to aid them in their work to match doctors to patients using the application. If the clinic administrator insist on not having a specialization, they can put in a generic value such as "GP" for general practitioner into the parameter and carry on without the application fatally crashing.

To keep the UG succinct and striaghtforward, we decided to omit talking about such corner cases especially because there could be many of them. If they do not adhere to the application's design, then an error message will be given with the correct action for the user to take to rectify the issue.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Even for any new clinic administrators trying to use the CLI application, they will refer to the UG, especially if the clinic expands to have more branches. Thus a bit more clarity in the UG would prove helpful. Also there is no mention of the default doctor being the GP and is not something the administrator should not be made to assume that this is the general value to input. 

To keep the UG succinct and striaghtforward, we decided to omit talking about such corner cases especially because there could be many of them. If they do not adhere to the application's design, then an error message will be given with the correct action for the user to take to rectify the issue.

Also, there doesnt seem to be the error message implemented as claimed by the HalpMi Team.