search

sogggy / pe

0 stars 0 forks source link

Improve error messages for invalid indexes #3

Open sogggy opened 3 years ago

sogggy commented 3 years ago

Perhaps there can be better error messages given by the application when inputting wrong parameters, such as invalid indexes.

image.png

image.png

When index is either negative or exceeded, both just says invalid, which is unclear.

image.png

In the last screenshot, the command format is correct, but it says that the command format is wrong.

nus-pe-bot commented 3 years ago

Team's Response

The command format that the tester is using is incorrect. Error message shown is correct as intended.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]