ShawnYip-Au / pe

0 stars 0 forks source link

Inconsistent Numbering #2

Open ShawnYip-Au opened 1 year ago

ShawnYip-Au commented 1 year ago

To reproduce bug

add patient > delete patient > add patient

patient add --n John Doe --p 98765432 --d 2001-03-19 --b B+ --a catfur --a pollen --v covax patient delete

Screenshot 2023-04-14 at 2.20.29 PM.png

nus-pe-bot commented 1 year ago

Team's Response

In our UG: https://ay2223s2-cs2103-f11-3.github.io/tp/UserGuide.html#patient---patient-functionalities

A patient is uniquely identified by a generated PATIENT_ID. Duplicate patients are allowed in the system as there might be patients with the exact same attributes that want to be vaccinated.

The PATIENT_ID is unique to a patient, it is not an index of the patients in the list. So if the patient has been removed from VMS, the removed patient's ID should not be reused by a new patient.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]