search

btricec / pe

0 stars 0 forks source link

More descriptive error message #2

Open btricec opened 4 years ago

btricec commented 4 years ago

image.png

The error messages for incorrect input is very generic. It took me very long to find my error in this input. Especially since there are so many parameters for the command, a specific error message pointing at which part of the input is incorrect would be helpful and make the app more user friendly, especially for beginners.

nus-pe-bot commented 4 years ago

Team's Response

Thank you for your insightful response. After much discussion, we agree that we could be more specific in our error messages. Thanks.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: I think that this bug has a low severity and not a very low severity. From the CS2113 website, it is stated that VeryLow severity is "A flaw that is purely cosmetic and does not affect usage e.g., a typo/spacing/layout/color/font issues in the docs or the UI that doesn't affect usage."

However, I feel that this isn't a purely cosmetic flaw. I found in extremely frustrating when my command kept failing over and over and having no idea why as it was difficult to spot the mistake in the entire line of text. I feel that this generic error message definitely affects user-friendliness, especially for a new user.