joshuan98 / pe

0 stars 0 forks source link

Multiple errors in command #24

Open joshuan98 opened 1 year ago

joshuan98 commented 1 year ago

In the event that there are multiple errors with the command, the group can consider displaying all invalid error messages so that the user needed not have to run two commands to find out that there are two errors.

image.png

nus-se-bot commented 1 year ago

Team's Response

The app is designed to help users fix their commands incrementally rather than dumping many errors at the user.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Perhaps I should have labelled this as a feature flaw but the group can consider this in future iterations. I would be content if the group considers this bug "NotInScope". Severity-wise, this would either be veryLow or Low. The team's line of reasoning would demand the user to continuously provide erroneous input until all mistakes are fixed, which could be easily avoided by providing all errors at once. Furthermore, the number of errors would only depend on the number of parameters in the command. Since the number of parameters is 3 in the screenshot, I do not see the harm in informing the user of all 3 errors at once.