search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

ug: add does not specify conditions of name uniqueness #4618

Open nus-se-script opened 5 months ago

nus-se-script commented 5 months ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

in your app, add n/John Doe m/A1234567Z e/johnd@u.nus.edu tl/johndoe01 t/cs2109s t/G19Group1 and add n/John Doe m/A1234567Z e/johnd@u.nus.edu tl/johndoe01 t/cs2109s t/G19Group1

can both run successfully in succession. ( because of the additional spaces in name)

As a user, i would expect these names to be considered unique, but the UG does not tell me what is considered unique and what is not

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

thewongdylan commented 5 months ago

Team's Response

Planned enhancements state that UUID is to be changed to matric number instead of name. Hence duplicate detection for names will not be needed in the future.

In the future, in the event that there are duplicate names, if matric numbers provided are the same, an error will be thrown since duplicate detection will be handled by the matric number.

Furthermore, if duplicate names are provided with different matric numbers, they will be accepted as 2 separate individuals with the same name, since their (unique) matric numbers are different.

Duplicate status (if any):

--