cleowenxuan / pe

0 stars 0 forks source link

Error command should be more specific #3

Open cleowenxuan opened 6 months ago

cleowenxuan commented 6 months ago

Instead of mentioning two errors such as "should not contain alphanumeric characters and spaces" and "should not be blank", the error given should be more specific and only write the first error message. This means that "should not be blank" should not be mentioned.

Screenshot 2024-04-19 at 4.40.54 PM.png

nus-pe-script commented 6 months ago

Team's Response

While limiting our error message further would better address this one use case, we believe that the error message, with its role to dictate the required formatting to the user, should encapsulate the full suite of formatting restraints on the name field.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** This feature flaw is of `Low` severity and not `VeryLow` because it displays error that is not specific enough and is not a cosmetic issue.