nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Allowing duplication of tutors of the same name and phone number but different emails for add person command #2372

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Tutors with the same name and phone numbers should be the same person but it is possible they have multiple emails and should not be counted as different people.

image.png

image.png

Steps to reproduce:

Step 1: type "add tutor n/Betsy Crowe p/1234567 e/betsycrowe@example.com a/Newgate st, block 123, #01-01 q/MSc, Master of Science i/National University of Singapore

/mostLiked"

Step 2: type "add tutor n/Betsy Crowe p/1234567 e/betowe@example.com a/Newgate st, block 123, #01-01 q/MSc, Master of Science i/National University of Singapore

/mostLiked"


[original: nus-cs2103-AY2223S1/pe-interim#1991] [original labels: type.FeatureFlaw severity.Low]

totsukatomofumi commented 1 year ago

Team's Response

We only see duplicate entries as having all the same particulars. This is to allow for flexibility incase there are people with the same name and same phone number, etc. This was our design choice.

Duplicate status (if any):

--