search

tzhan98 / pe

0 stars 0 forks source link

Error that shows a duplicate entry of `nric`, `email` and `phone number` should also show index of duplicate entry #7

Open tzhan98 opened 2 years ago

tzhan98 commented 2 years ago

If user tries to add a user with valid nric of S1234567B, and warning The edited NRIC of the applicant already exists appears, the user will have to scroll though potentially 1000 contacts (stated in the NFR) to search for the entry with the duplicate nric of S1234567B. This is because the nric of the duplicated entry of the old contact could be the contact that was mistyped.

This is also because there is no way of searching or filtering by nric.

soc-pe-bot commented 2 years ago

Team's Response

It is more of a suggestion, to make the life of the user easier. But this is not technically a feature flaw, as the error message is correct in saying that the duplicate field exists.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Since it is more of a suggestion, should It be a not in scope instead of rejected?