freddychenyouren2 / pe

0 stars 0 forks source link

Email need not contain period "." #4

Open freddychenyouren2 opened 7 months ago

freddychenyouren2 commented 7 months ago

Users may sometimes forget to write the ".com" behind the email field of the contact. This issue is to precisely test that:

The following command is done add n/123 p/999 e/email@yes

Expected: Error message for Email field to require "." period Actual: Email without "." period is allowed, as shown in the screenshot below.

image.png

nus-pe-script commented 7 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Thank you for testing this! However, emails may not always contain periods! Please search up "dotless domains".

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]