search

shaunlxw / pe

0 stars 0 forks source link

UG add command: constraints of email #19

Open shaunlxw opened 4 months ago

shaunlxw commented 4 months ago

Constraints of email should be more descriptive and reflect what will be shown in the error message when user enters the email in invalid format.

image.png

image.png

nus-se-script commented 4 months ago

Team's Response

Severity changed to VeryLow since it does not affect the usage for a normal user and the app displays a more descriptive error message for the user.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: It is not purely cosmetic, and does bring minor inconvenience in rare situations. Common sequence of actions for a user is:

  1. Read UG
  2. Use command

Without giving the full constraints of the email field in the UG, users could enter the email in the invalid format, and will be met with the error message, bringing inconvenience as the efficiency is significantly reduced when they have to fix the command.