nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Duplicate patients allowed #3182

Open nus-se-script opened 11 months ago

nus-se-script commented 11 months ago

2 patients of the same name are allowed to be added into care central. This might cause a duplicate in records of a patient.

steps: 1) type in "add-patient n/john doe ic/S0123456A a/45 p/12341234 e/johndoe@example.com"

2) type in "add-patient n/John Doe ic/S0123456A a/45 p/12341234 e/johndoe@example.com"

you can see that both are recorded in the care central app in the screen shot below: Screenshot 2023-11-17 at 4.17.11 PM.png

This might cause duplication of patients and when updating the patient records, might not capture all the details in 1 person. 1 way around this might be to detect names without a case sensitive approach such that a patient with the same name will be rejected (or given a warning)


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

tanboonkhong commented 11 months ago

Team's Response

  1. Patient of the same name can exist and we treat it as two different person, it is not uncommon in the real world to have two person that shares a same name, disallowing patients of the same name would be inconvenience for the user.

  2. We plan to implement unique NRIC validation to distinguish two unique patients as our planned enhancement. Please refer to Appendix: Planned Enhancement for more details.

Duplicate status (if any):

--