nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

UG does not indicate restrictions in adding duplicate person names but application does. #1073

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

Summary

Trying to add someone sharing the same name as an existing person results in an error, even though the UG does not mention such restrictions.

add command

image.png

parameter constraint

image.png

Steps to reproduce

  1. Ensure person Alex Yeoh exists in list
  2. Enter add n/Alex Yeoh

Expected

Duplicates allowed and command succeeds.

Observed

Error given due to duplicate contact.

image.png

Tester Information

System Information:

Java version:


[original: nus-cs2103-AY2425S1/pe-interim#454] [original labels: severity.Low type.DocumentationBug]

flyingsalsa commented 1 week ago

Team's Response

If you look at the screenshot you sent for parameter constraints it does actually mention that duplicate names are not allowed, just that they are case sensitive.

image.png

Duplicate status (if any):

--