nus-tic4002-AY2122S2 / pe-dev-response

0 stars 0 forks source link

Edit Command issue #4 #189

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

Steps to reproduce:

  1. edit 1 n/John p/900114232 e/john@example.com a/621 Bedok Rd, 470623 t/colleague

Actual:

image.png

Expected: Validation check for phone number, assuming we are only doing singapore based numbers. Else could add country code as a validation check


[original: nus-tic4002-AY2122s2/pe-interim#174] [original labels: severity.Medium type.FunctionalityBug]

l-shihao commented 2 years ago

Team's Response

assuming we are only doing singapore based numbers

we don't assume that, it could support China phone number with 11 digitals / Japan phone number with 10 digitals

Duplicate status (if any):

--