nus-cs2103-AY2021S1 / pe-dev-response

0 stars 0 forks source link

Unrealistic address accepted #4590

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

"p-add n/Elon Musk g/MALE bd/2018-12-27 bt/A+ p/98765432 e/johnd@example.com a/9999999999999" is accepted. Suggestion to make it a bit more foolproof is to either accept a postal code with less digits than this, or make the address have alphabetical characters.

image.png


[original: nus-cs2103-AY2021S1/pe-interim#4408]

yeohhq commented 3 years ago

Team's Response

As address inputs are quite varied across the whole world (there are no fixed way to show an address), we have decided to not put any restrictions on the address field.

Your suggestion for:

Nonetheless, thank you for the suggestions!

Duplicate status (if any):

--