carriezhengjr / pe

0 stars 0 forks source link

Case-sensitivity should not be allowed for adding a person #4

Open carriezhengjr opened 1 year ago

carriezhengjr commented 1 year ago

After entering the command add -p n/John Doe e/johnd@example.com p/98765432 l/1 c/Meta to add a person named John Doe, another person with exactly the same detail but with the name in different cases can be added using the command add -p n/john Doe e/johnd@example.com p/98765432 l/1 c/Meta.

Screenshot 2022-11-11 at 16.29.36.png

Since all the details are the same, it should be considered a duplicate (unlike the case where 2 people have the same name, but different personal particulars like email etc, so they are different people). Case sensitivity of the same identity should not be allowed in the App.

nus-se-bot commented 1 year ago

Team's Response

image.png

This is addressed in section 6, Planned updates, of the UG. We acknowledge and plan to fix the issue in a future update, but for v1.4 it is out of scope.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]