bug: email format inconsistency between user guide and application

[wxiaoyun]

Description

• User guide stated that:

The local-part should only contain alphanumeric characters and these special characters: +_.- .

• However, when adding a contact with emails that has consecutive "." (e.g. test..valid@email.com), the email is rejected.

Steps to reproduce

• Run this command: add /com Google /desc test /status to_apply /poc Test Name2 /email test..valid@email.com /phone 99999999

Expected

• Success command execution

Actual

• Command rejected due to invalid email format

Potential Suggestions

• Update expected format of email in user guide or email validation logic

Screenshots

[soc-pe-bot]

Team's Response

This works as intended, because emails cannot have consecutive periods in the real world. Suppose this is a typo by the user, rejecting it is expected.

It would be too tedious to include all possible edge cases of email validation for the local part (consecutive periods, registered domain name etc..). Thus, we put this as NotInScope, since to include all possible edge cases in the UG has a lower priority than the work already done in v1.4, the email validation works as intended, and for the benefit of the user.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

• [ ] I disagree

Reason for disagreement: [replace this with your explanation]

---