search

andreatanky / pe

0 stars 0 forks source link

Error message has an incorrect format #3

Open andreatanky opened 3 years ago

andreatanky commented 3 years ago

The UG mentions that items 4 and 7 (email and allergies) of patients are optional. However, the error message as shown in the screenshot below doesn't have email field in square brackets. (both email and allergies should have brackets)

Screenshot 2020-11-13 at 4.31.19 PM.png

Screenshot 2020-11-13 at 4.23.52 PM.png

nus-se-bot commented 3 years ago

Team's Response

We agree that we should have included square brackets in the error message, but it should not affect the fuctionality of the command greatly.

Items for the Tester to Verify

:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

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

Reason for disagreement: Screenshot 2020-11-18 at 4.57.02 PM.png

In the CS2103T website, severity.VeryLow are for flaws that are purely cosmetic and does not effect usage. However, the inconsistency of square brackets in the error message can lead to confusion and repeated wrong inputs as the user tries to rectify his mistakes using the error message.

I feel that it fits the description of severity.Medium best as this might cause occasional inconvenience to some users who choose to rectify their mistakes using the error message in the GUI.