nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Use case extensions #5790

Open nus-se-bot opened 3 years ago

nus-se-bot commented 3 years 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.


Here is the use case for adding a tutor:

image.png

The extension does not account for the case in which a user with the same phone number (i.e., a duplicate is trying to be added) already exists in the database. This should be a relevant extension to this process.

Similarly for adding a student as well, it does not account for phone number already existing. Not creating a separate bug report as it is the same issue present in multiple places.

As this is quite important, the severity.Low is applied rather than severity.VeryLow.


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

knotstoks commented 3 years ago

Team's Response

Hi, thank you for your suggestion!

This is a repeated issue as #5792 except it is for Add Command.

Duplicate status (if any):

Duplicate of #5792