nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Phone Number Format #4992

Open nus-pe-bot opened 2 months ago

nus-pe-bot commented 2 months ago

Screen Shot 2567-04-19 at 16.49.30.png

Screen Shot 2567-04-19 at 16.50.55.png

I saw in the DG talking about consideration of limiting number of character to 3 -15 which I assumed that this is meant for recording non SG phone number as well.

case 1: Assuming allowing non SG phone

case 2: Only SG phone since all are NUS student


[original: nus-cs2103-AY2324S2/pe-interim#4568] [original labels: severity.Medium type.FeatureFlaw]

kaitinghh commented 2 months ago

Team's Response

Our app actually considers a case 3:

Our app accommodates students and TAs. It is reasonable that TAs can have office landlines which can have variable number lengths. Therefore, we want to accept a variable number length instead of just 8 digit Singaporean phone numbers.

Duplicate status (if any):

--