nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Add command does not check for duplicate phone numbers #1809

Open nus-pe-bot opened 1 day ago

nus-pe-bot commented 1 day ago

Currently, the add command allows users to add patients with the same phone numbers. In a real world worst-case scenario, if these patients only provided their basic information, there would be no way to contact one of the patients without one of the patients themselves first editing their numbers to be unique, which would be quite problematic for a health service.

image.png

image.png


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

yugan01 commented 21 hours ago

Team's Response

We reject this bug as this is an intended behavior of our app. There is a possible situation of where 2 different patients may have the same phone number in real life. For e.g. 2 children registered as unique patients in the system have their parent's contact number as the phone number in the system.

Even though patients have a next of kin and the NOK phone number included, this is intended to provide a second point of contact in emergency situations if the primary contact cannot be reached. This is also how it is typically implemented in healthcare systems in real life.

Duplicate status (if any):

--