Xilef121 / pe

0 stars 0 forks source link

Add command accepting invalid emails #6

Open Xilef121 opened 6 months ago

Xilef121 commented 6 months ago

Screenshot 2024-04-19 at 4.40.55 PM.png Add command allows for invalid emails which do not contain the "."

soc-pe-bot commented 6 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Misleading error message

Screenshot 2024-04-19 at 4.18.04 PM.png

The email test..test@test.com fully satisfies the constraints mentioned but is not being accepted.


[original: nus-cs2103-AY2324S2/pe-interim#1786] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Accepted as is.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: This would not be a duplicate to the earlier bug i submitted regarding misleading error message. Reason being, one is the constraints of email not being stated well enough in the error message, while this issue is regarding the accepting of invalid emails. Both being different issues