Non-specific error message for invalid emails

Steps to reproduce:

Enter this command: "add n/James Ho p/22224444 e/ @example.com a/123,Clementi Rd, 1234665 t/friend t/colleague"
Error message shown: Emails should be of the format local-part@domain and adhere to the following constraints:
The local-part should only contain alphanumeric characters and these special characters, excluding the parentheses, (+_.-). The local-part may not start or end with any special characters.
This is followed by a '@' and then a domain name. The domain name is made up of domain labels separated by periods. The domain name must:
- end with a domain label at least 2 characters long
- have each domain label start and end with alphanumeric characters
- have each domain label consist of alphanumeric characters, separated only by hyphens, if any.

Expected: An error message specific to the invalid email (missing local-part) should be displayed. Actual: A long string of error message that encompasses numerous potential issues

Good catch on the invalid email! However, I would like to suggest this improvement on the error message shown:

Users might unknowingly leave out the local-part when keying in their email. However, as pasted above, the error message is very very long as it encompasses many different possible mistakes the user has made
It is extremely tedious for the user to check them one by one to identify the error they made in their email.
My suggestion would be to split the validation into different stages (e.g. check local-part, throw local-part error if local-part is missing etc.)

yuechen2001 / pe

Non-specific error message for invalid emails #6

Team's Response

Items for the Tester to Verify

:question: Issue response