search

nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Application allows user to input placeholder nric of patient, but doesnt allow the nric to be changed. #3002

Open nus-se-script opened 11 months ago

nus-se-script commented 11 months ago

User will have to go through an extra step of copying all the details/records of the existing patient, then deleting the existing record, then creating a new patient/record. This could be very troublesome if the existing patient has a lot of records.

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

ryanongwx commented 11 months ago

Team's Response

As patient's NRICs are not expected to change, we did not feel a need to allow the editing of NRIC to further safeguard its use as a unique ID for each individual. We acknowledge that an erroneous NRIC input can happen on patient creation. However, in such a scenario there would be no existing records to have to recreate and hence not much inconvenience. However, we do acknowledge that this would be a nice to have enhancement in the future and hence choose to respond with NotInScope.

Duplicate status (if any):

--