nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Deleting a person using NRIC is not as convenient as using index #3517

Open nus-se-script opened 6 months ago

nus-se-script commented 6 months ago

Currently, the app supports deletion of a person using NRIC. However, it is not convenient as the user will need to type the NRIC accurately which contains both alphabets and numbers. Perhaps using the index for each person is easier and faster as it is also unique like NRIC.

To add on, the user cannot copy the NRIC shown in the contact card which leaves no choice but to type out the IC character by character.


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

limrik commented 6 months ago

Team's Response

Thank you for your suggestion!

Our team has actually considered using index as well.

While the use of index may improve usability, it may lead to detrimental effects such as deleting the wrong patient's medical records or adding/removing medical allergies to the wrong patient. Hence, it was a deliberate choice to use NRIC despite it being slower as we have deemed that accuracy is more important as compared to speed in the healthcare sector. We have also included this in our FAQ section.

Screenshot 2024-04-21 at 12.19.05 AM.png

Furthermore, we have taken precautions to prevent editing of the NRIC field of the patient, making it secure and suitable for use as a unique identifier in our commands.

Screenshot 2024-04-21 at 12.19.19 AM.png

Duplicate status (if any):

--