search

saiutkarsh33 / pe

0 stars 0 forks source link

Can edit a person's email to be one that is already existing in the contact book. #5

Open saiutkarsh33 opened 6 months ago

saiutkarsh33 commented 6 months ago

Context

A feature in this app is the ability to edit email addresses. An email address of a person can be edited to be the same as the email address of a previous person

Expected Behaviour vs Actual Behaviour

It is expected that 2 people do not share one email address, especially in our age of internet communication, therefore an error should be thrown, but it is allowed.

How to replicate

Have at least 2 persons. Edit the email address using the edit keyword of the second person, using the first person's email address. edit 2 e/[phone_number_of_first]

Why this severity:

These will usually be typos, and therefore will not happen often. However, it would be good to have this feature to avoid such typos.

nus-pe-script commented 6 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

This is an intended feature as multiple people will be able to share the same email address in your contact list.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]