AY2324S2-CS2103T-W10-2 / tp

https://nus-cs2103-ay2324s2.github.io/tp/
MIT License
1 stars 5 forks source link

[PE-D][Tester A] Email can be added without root domain extension #264

Closed nus-pe-bot closed 6 months ago

nus-pe-bot commented 6 months ago

To replicate:

/add-supplier ; name : PetCo ; phone : 98673098 ; address : Meow Street 24 ; email : ilovewombatstoo@gmail ; product : kibble ; price : $98/bag

Screenshot 2024-04-05 at 5.25.56 PM.png

Suggestion:

Validate the email with the presence of a "." after the "@"


Labels: type.FeatureFlaw severity.Low original: belligerentbeagle/ped#9

yleeyilin commented 6 months ago

We purposefully accepted emails with no root extension since there exists emails without these root extensions. You can refer to the following link for email examples:

image

https://en.wikipedia.org/wiki/Email_address#Examples