search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Different format used for the UML #59

Open nus-se-script opened 2 months ago

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

The plantUML format used here differs from the format used by CS2113. The green circles to represent the classes should be removed to align with the CS2113 format

Screenshot 2024-04-19 173007.png

Screenshot 2024-04-19 173201.png

[original: nus-cs2113-AY2324S2/pe-interim#58] [original labels: type.DocumentationBug severity.VeryLow]

EugeneChanJiajun commented 2 months ago

Team's Response

We understand that its slightly different from the textbook. However, there is no statement in the textbook that says the circles should not be present and the TA did not highlight this as an error during her review of the diagram.

Duplicate status (if any):

--