search

sugiyem / pe

0 stars 0 forks source link

Phone validation bug #6

Open sugiyem opened 2 years ago

sugiyem commented 2 years ago

The system allow invalid phone number, such as shown below, to be inserted.

image.png

nus-pe-bot commented 2 years ago

Team's Response

This is not a bug, but a feature as mentioned in the UserGuide under Student Manager Command Parameters.

image.png

If we were overzealous with our input validation, such as limiting to 8 numbers some phone numbers with longer length might not be included. For example, those from other countries with extra long numbers might bot be able to be added into out application. As such, we decided to only accept numbers with length greater than 3.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]