bobscodedump / pe

0 stars 0 forks source link

Can add 2 different people with the same email #4

Open bobscodedump opened 11 months ago

bobscodedump commented 11 months ago

Screenshot 2023-11-17 at 4.21.09 PM.png

Since an email is unique to an individual, it may be best to disallow duplicate emails.

nus-se-bot commented 11 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]

Different people are able to have the same email

While the app does not allow 2 people to have the same, they are able to have the email as well as other details.

image.png Steps:

  1. add n/John Doe p/98765432 e/johndoe@gmail.com a/John street, block 123, #01-01
  2. add n/John p/987654321 e/johndoe@gmail.com a/John street, block 123, #01-01

[original: nus-cs2103-AY2324S1/pe-interim#3860] [original labels: type.FeatureFlaw severity.Medium]

Their Response to the 'Original' Bug

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

Thank you for raising this issue. If you have carefully read our planned enhancements, we have clearly indicated that in the future iteration, we will be working towards making a person unique in CampusConnect. However, as of now, due to time constraints, we forgo this current problem we noticed.

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: [replace this with your explanation]


## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]