search

nus-cs2103-AY2021S1 / pe-dev-response

0 stars 0 forks source link

Large architecture diagram with too many details #3396

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.

The architecture diagram for the Logic component is very large and the text is very small, making is hard to read. Some details may not be necessary for an architecture diagram (i.e. specific ItemXYZCommand, HelpXYZCommand, notes, etc), or would be better included as a seperate diagram.

image.png

[original: nus-cs2103-AY2021S1/pe-interim#3125]

Wincenttjoi commented 3 years ago

Team's Response

The diagram above is a Class diagram, which the tester has mistaken as an architecture diagram. This level of detail is appropriate for a Class diagram.

Duplicate status (if any):

--