nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Leading zeros error not detected, generic invalid format error message #1656

Open nus-se-script opened 6 months ago

nus-se-script commented 6 months ago

While stated in the UG that leading zeros are required, the input validation does not check for this error and instead returns a generic, non-specific invalid command format error.

image.png

severity.Medium - This could particularly inconvenience users as they do not know what is wrong with their input since omitting leading zeros still seems logical.


[original: nus-cs2103-AY2324S2/pe-interim#2089] [original labels: severity.Medium type.FeatureFlaw]

LWS49 commented 6 months ago

Team's Response

Our error message clearly explains the error in this context. Users would be able to understand that leading zeros are necessary after seeing the error message. Hence, we do not believe that a more specific error message is necessary, and that users would not know what is wrong with their input without a more specific error message.

Duplicate status (if any):

--