nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Phone prefix inconsistency #2793

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

When users run find all p/John the output box shows:

Screenshot 2024-11-15 at 5.08.41 PM.png

This behaviour of p/ for find different from how it is in other commands, for example when running edit 1 p/a9990000 in the main window:

Screenshot 2024-11-15 at 5.15.29 PM.png

The team might want to consider to incorporate the check for value of p/ and throw an error when the input given by user does not adhere to "only contain numbers, and it should be at least 3 digits long". Currently the find command can execute as usual without error, it would be nice to inform users by displaying error message.


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

thortol commented 1 week ago

Team's Response

No details provided by team.

Duplicate status (if any):

--