search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Add command duplicate checking #5622

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

image.png

Tried to add a tutee with the same name but different numbers (which is very likely in the real world) but the app registers is as the same person. I think it would be better if the app takes into consideration phone numbers when doing duplicate checking

Steps to replicate

  1. add n/John Doe p/93456789 sch/Temasek Jc l/j2 a/135 Rivervale Link
  2. add n/John Doe p/98765432 sch/Temasek Jc l/j2 a/135 Rivervale Link

[original: nus-cs2103-AY2122S1/pe-interim#5645] [original labels: type.FeatureFlaw severity.Medium]

nhjryan commented 2 years ago

Team's Response

Similar issue (Multiple buggy behaviors that are actually caused by the same defect and cannot be fixed independently, i.e. the equals method for Tutee).

Duplicate status (if any):

Duplicate of #4469