search

nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Model class diagram unecessarily massive #2375

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year 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

Person class has so many attribute classes shown here. Displaying them here is unnecessary.

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

leecaregene commented 1 year ago

Team's Response

We argue that displaying these attribute classes is not unnecessary. Omitting them from this class diagram might impede understanding of the model component instead.

Duplicate status (if any):

--