search

jiaweiteo / pe

0 stars 0 forks source link

Invalid email format accepted #4

Open jiaweiteo opened 3 years ago

jiaweiteo commented 3 years ago

Command: add ct/s n/hi p/0123 e/asd@com

Expected: Error message for invalid email since email needs to have a domain followed by the address.

Actual: Input accepted.

image.png

nus-se-bot commented 3 years ago

Team's Response

This issue is inherited from AB3 as seen replicated below. According to forum issue #429, this bug report is rejected since it's the same behavior as in AB3.

Address, phone and email validation have been inherited from AB3 and they are for typical use cases. Users are not restricted to certain address/phone/email to cater to wider target audience which may need to handle foreign address or phone. Unnecessary restriction may lead to it being bugged/unusable for some of our intended target audience.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]