nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Inconsistent error message for delete command when format is right #6004

Open nus-se-script opened 1 year ago

nus-se-script commented 1 year ago

image.png Since delete contact 1 follows the correct format, the app should not output the message of invalid format. Instead, it should warn that the index is not valid.


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

Nixx162 commented 1 year ago

Team's Response

image.png

As given in our UG, CONTACT_ID needs to be a positive integer. Since 0 is not a positive integer, this is indeed not the correct format.

Duplicate status (if any):

--