Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No validation for invalid email address #3

Open
Linxcathyyy opened this issue Nov 12, 2021 · 1 comment
Open

No validation for invalid email address #3

Linxcathyyy opened this issue Nov 12, 2021 · 1 comment

Comments

@Linxcathyyy
Copy link
Owner

Linxcathyyy commented Nov 12, 2021

Screenshot 2021-11-12 at 4.27.41 PM.png

Screenshot 2021-11-12 at 4.31.37 PM.png

I am able to add an invalid email address.

@nus-pe-bot
Copy link

nus-pe-bot commented Nov 16, 2021

Team's Response

Accepted, but the severity should be very low, because this case only happens if someone intentionally inputs invalid emails.

Items for the Tester to Verify

❓ Issue severity

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

  • I disagree

Reason for disagreement: I don't think this is a purely cosmetic issue as it does affect usage. Your team mentioned that "this case only happens if someone intentionally inputs invalid emails". If the manager accidentally enters an invalid email but the app allows it, this will cause inconvenience in the future when the manager wants to email this particular client/employee/supplier. Hence this case appears only in very rare situations and causes a minor inconvenience only. Therefore, the severity should be low.


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants