search

crypto-code / pe

0 stars 0 forks source link

Add command error message #2

Open crypto-code opened 2 years ago

crypto-code commented 2 years ago

If we run the following command, add n/John Doe m/CS2100 t/Lab04

It prints out an invalid command format message.

image.png

Although here the format is correct, just that the required field is missing.

nus-pe-bot commented 2 years ago

Team's Response

The invalid command message is indeed correct. A proper call to add requires the name, module, and email fields to be provided.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I agree with decreasing the severity as it does not affect the use of the app as much and causes a minor incipience in rare cases. But at the same time, a user would expect a more specific error message especially in cases when there are multiple tags included and the entire command is not visible. Imagine this command executed.

image.png

This would cause a minor inconvenience to a user as they have to spend time figuring out what the issue is and it may simply be that they forgot a specific required parameter.

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]