search

zhengster / pe

0 stars 0 forks source link

"Add medicine" command (or commands in general): if user accidentally omits a comma, perhaps include a more specific error message #14

Open zhengster opened 2 years ago

zhengster commented 2 years ago

image.png In this case, the user did enter all parameters, but forgot the comma in between "A123" and "Paracetamal." The program thinks that A123 and Paracetamal are conjoined/part of the the MedicineId, and thus thinks one or more parameters are missing. Perhaps you could tell the user that, instead, "There may be on or more parameters that are missing. Also ensure that all arguments are separated by a comma."

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, we could add a more specific message for the user but that would be for future development. It was stated in our documents how to properly add medicine. Severity is Low not medium

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your reason]