AY2324S1-CS2103T-T08-1 / tp

MIT License
0 stars 5 forks source link

Phone Number Bug: Not restrictive enough #196

Closed yanghengtang closed 10 months ago

yanghengtang commented 10 months ago

Current implementation for regex is to check if length >= 3 and all characters is number.

This allows phone number shorter than 8 character to be entered:

Screenshot 2023-11-03 at 1 01 01 PM

As well as phone number with more than 8 characters:

Screenshot 2023-11-03 at 1 02 08 PM

Let's restrict it to the Singapore context