AY2425S1-CS2103T-F08-3 / tp

MIT License
0 stars 5 forks source link

[PE-D][Tester D] find command using email #208

Closed nus-pe-script closed 5 days ago

nus-pe-script commented 1 week ago

For the input find e/char, the command is unable to match this to the email charlotte@example.com. The error message suggests that the email must follow the format to be considered valid, but limit how useful the find command is since it is unlikely for user to remember full email of contact when trying to find contact via email.

image.png

Can consider doing partial matching for the find command for the email field (i.e. char would match charlotte@example.com). If not, it would be good to specify in user guide that find command for email field only does exact string matching to avoid confusion of user.


Labels: type.FeatureFlaw severity.Medium original: kngys/ped#17

PeterHW963 commented 1 week ago

PLanned enhancement possibly. NVM Just edit UG

psh12320 commented 1 week ago

Update UG to state that find command with e/ prefix only accepts valid emails