search

nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Employee missing composition for EmployeeName #3734

Open nus-se-script opened 9 months ago

nus-se-script commented 9 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.

image.png

Employee missing composition for EmployeeName, should contain composition symbol.

[original: nus-cs2103-AY2324S1/pe-interim#5154] [original labels: type.DocumentationBug severity.VeryLow]

XihuaZ commented 9 months ago

Team's Response

Hi, there should not be composition symbol since the EmployeeName objects for superiors come from other Employee objects instead. Hence, deleting this Employee should not cause the other EmployeeName to be deleted since they below to other Employee objects.

Duplicate status (if any):

--