Class diagrams do not comply to what we have learnt

[woozije]

The icons beside the classes, method names as well as the way the classes are grouped together in the class diagrams do not seem to comply to the class diagrams we have learnt in class. Perhaps a legend or note to refer to the different icons might be easiser for the user when reading it.

[nus-pe-bot]

Team's Response

since we used plantuml to draw our diagrams there are limitations to the tool, you could refer to their website for the syntax (https://plantuml.com/class-diagram)

here's the visibility syntax table as an example:

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

• [x] I disagree

Reason for disagreement: Referring to Prof's tip in github forum issue 112, it was stated that this is considered non-standard UML notation. In addition, there is a way to turn off the icons as well.

---

:question: Issue severity

Team chose [severity.High] Originally [severity.Medium]

• [x] I disagree

Reason for disagreement: Severity may not be that high as it is still readable and does not cause a major problem.

---