search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Incorrect error message displayed #560

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

Input: edit 2 t/ p/99999999 0/

The above input has a valid phone number 99999999 as specified in the UG and app as seen in the error message below. The error in this input is the addition of 0/ but the error message claims it is the phone number which is not the case as explained previously.

It would be good to change the validation/parsing to correctly pinpoint 0/ as the error and not the phone number so the user is not confused what caused the command to not work.

image.png

[original: nus-cs2103-AY2425S1/pe-interim#790] [original labels: type.FeatureFlaw severity.Low]

LemonDrew commented 1 week ago

Team's Response

There is no '0/' as part of our current prefixes. Hence, the p/ sees the input as "99999999 0/", therefore the application gives the error message which is correct message.

Duplicate status (if any):

--