search

ruishanteo / pe

0 stars 0 forks source link

find p/ non-numerical inputs #14

Open ruishanteo opened 9 months ago

ruishanteo commented 9 months ago

find p/ accepts non-numerical inputs which is contrasting to the phone number constraint of only accepting numerical inputs.

find p/asd123 should not be accepted

nus-se-bot commented 9 months ago

Team's Response

Thanks for your suggestion tester! The "Find" feature is meant to be as flexible as possible. Thus it allows you to enter anything to search. This does not affect the user, as the user would simply find no results. Overzealous input validation for the "Find" feature would make it quickly very annoying for the user.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]