syj02 / pe

0 stars 0 forks source link

Unclear exception thrown for typos in command #9

Open syj02 opened 2 months ago

syj02 commented 2 months ago

Rather than just printing "Invalid command", it might be better to also list all the commands or highlight where the error is. In this case, "addmaintrips" was written instead of "addmaintrip", it is not a very prominent mistake and could take awhile to find, yet the exception thrown is not very helpful with identifying a small mistake like this.

Screenshot 2024-04-19 at 16.55.20.png

nus-pe-script commented 2 months ago

Team's Response

Still is an invalid command, but this suggestion is a possible feature that could be implemented. Though not a bug

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]