maguireong / pe

0 stars 0 forks source link

editvisit feature: Incorrect parameters in error message #5

Open maguireong opened 3 years ago

maguireong commented 3 years ago

A legitimate user behavior is not handled. Simply typing 'editvisit 1' prompts this error which is [PATIENT INDEX] i/[VISIT INDEX] parameters to be optional when they have to be specified as written in the 2nd image. Similar for editvisit error message

Screenshot 2020-11-13 at 4.34.49 PM.png

Screenshot 2020-11-13 at 4.35.39 PM.png

nus-pe-bot commented 3 years ago

Team's Response

Error message should be similar to the UG phrasing.

Reason for changing the severity to VeryLow: Comestic flaw in error message and does not affect the usage.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: I think that the severity should not be very low because the given error message shown in the command box does not work at all. Users would face this issue occasionally and they would not be able to use this feature if they blindly follow the instructions written in the command box instead of following the UG, which happened to me. Hence, I think that the severity should be medium.