takufunkai / pe

0 stars 0 forks source link

Inconsistency in commands #1

Open takufunkai opened 2 years ago

takufunkai commented 2 years ago

Problem: Some commands require to reference contacts by their NRIC (unique identifier), other commands require reference by index. There does not seem to be a standard/rationale behind it.

image.png

Adding medical information requires i/NRIC

image.png

Whereas we won't be able to edit the contact's medical information by NRIC, and only by index.

nus-se-bot commented 2 years ago

Team's Response

Yes, there is. When adding records, we have to specify which patient this record belongs to hence providing the NRIC is crucial. When editing, we use the index to refer to the record in the currently viewed screen. This is because users will likely not remember the information stored in each record, hence implementing edit to only work on the currently viewed screen makes sense so that users can refer to the record they want to edit.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]