search

chen-xi-cx / pe

0 stars 0 forks source link

Wrong error message for addmod command #2

Open chen-xi-cx opened 5 years ago

chen-xi-cx commented 5 years ago

the error message says invalid matriculation number (which do exist) but is the parameter is wrong (should be s/). This is not highlighted.

image.png

nus-pe-bot commented 4 years ago

Team's Response

Accepted. While the error message is technically correct: No valid student identifier is given ("/s" with index or matric code). However, the error message should be clearer and more specific. If a valid module and invalid student is given, it should clearly state what the error is instead of being ambiguous between matric id or index(for student class)