nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Names must be unique #2217

Open nus-se-script opened 7 months ago

nus-se-script commented 7 months ago

TutorRec does not allow for multiple persons with the same name.

In real life, there are common full names such as "Ryan Tan" or "Tan Wei Jie". Although a possible workaround will be naming them "Ryan Tan1" and Ryan Tan2", this will inconvenience the user and make it difficult to use.

To replicate, perform the following commands:

  1. add n/ Ryan Tan
  2. add n/ Ryan Tan

Behaviour: "This person already exists in the address book".


[original: nus-cs2103-AY2324S2/pe-interim#1821] [original labels: severity.Low type.FeatureFlaw]

tanqinyong commented 7 months ago

Team's Response

Fair point, but as primary school students there isn't many identifiers aside from the names. Thus, it has to be the unique identifier.

Duplicate status (if any):

--