Punpun1643 / pe

0 stars 0 forks source link

Error message for duplicate person not informative enough #4

Open Punpun1643 opened 2 years ago

Punpun1643 commented 2 years ago

BC81C08D-8F63-4334-81C0-4FFF82D42F0C_4_5005_c.jpeg

Explanation: Since your app allow for the add of person with the same name, when there are many persons in the list, user will not remember already about the details of the person that have been added. E.g. there may have so many Nicoles that the user may not remember which Nicoles have been added and which have yet to be added. In this case, error message for duplicate person should be like "there already exist person with this email address" etc.

Severity: Low

Justification: Because the user can just stop adding that person and try to find out which attributes are duplicated with another person in the list. Not verylow because it is not a cosmetic issue.

soc-se-bot commented 2 years ago

Team's Response

This error message only occurs when the user attempts to add an employee with attributes that match another employee's attributes exactly; as such, we do not consider the lack of specificity in the duplicate employee error message a feature flaw.

If we modify the duplicate employee error message following your suggestion, it would look something like this:

There already exist person with this name, phone number, email address, address, rate, and tags

Which practically conveys the same information as our existing duplicate employee error message (that all of the attributes match another employee's attributes), and makes it even more cumbersome for a user to read and understand.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]