danielleloh / pe

0 stars 0 forks source link

person with same name but different capitalisation are identifies as different people by the application. #3

Open danielleloh opened 4 months ago

danielleloh commented 4 months ago

steps to reproduce

  1. add person with the following: add n/cookie monster a/sesame street p/91234567 e/cookie.monster@example.com

  2. add another person with the following: add n/Cookie Monster a/sesame street p/91234567 e/cookie.monster@example.com

expected

since the application uniquely identifies same persons by name (as shown in screenshot 1),

the application should disallow the addition of second person with name Cookie Monster as a person with name cookie monster already exists within the application. However, cookie monster and Cookie Monster are likely the same person, so the application should output and error and disallow the addition.

actual

the application adds second person with capitalised name. (screenshot 2)

screenshots

Screenshot 2024-04-19 at 4.17.29 PM.png

Screenshot 2024-04-19 at 4.28.15 PM.png

nus-pe-script commented 4 months ago

Your response not required for this bug as the team has accepted the bug as it is.