nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Unstandardized notation in UML diagrams #3230

Open nus-se-script opened 6 months ago

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

Can consider indicating the multiplicity of "Name" and "Email" to make it consistent with other components (eg. Tag) in the diagram

Can standarize the composition/aggregation notations too!

_{[original: nus-cs2103-AY2324S2/pe-interim#2659] [original labels: severity.Low type.DocumentationBug]}

AjayArvind2207 commented 6 months ago

Team's Response

Thank you for the feedback! While we agree that it could be improved, we do not believe our UML diagram incorrectly describes the relationships between the classes.

nus-cs2103-AY2324S2 / pe-dev-response

Unstandardized notation in UML diagrams #3230

Team's Response

Duplicate status (if any):