Team's Response

The purpose of the diagram below is slightly unclear as it is not labelled with the title of the diagram.

This diagram is under the Command component section, and the purpose of the diagram should be clear from this - it describes the Command component.

The connection between the enumeration classes and the command classes could be indicated to clarify which commands access the enumeration class while executing.

There does not need to be any links between usage and the enumeration as long as it is stated in the attributes section of the class itself. https://nus-cs2113-ay2021s1.github.io/website/se-book-adapted/chapters/uml.html#enumerations

I believe the arrow that indicates inheritance could be a bit larger as the arrows right now look kind of like connecting arrows.

I believe you mean they look like the association arrows. However, it is clear that the arrow head is hollow.

Maybe the size of the diagram could be made larger for easy viewing

This is due to the conversion to PDF, but if this is referring to the font size, then this would be a duplicate of #1861

I think that an architechture diagram could have been a little bit better to represent the various commands rather than a UML diagram as there is less complexity involved

Thanks for the suggestion but without justification for how the architecture diagram would be better, I do not think I can accept this.

Items for the Tester to Verify

❓ Issue severity

Team chose [severity.VeryLow]
Originally [severity.Low]

• I disagree

Reason for disagreement: [replace this with your reason]

from pe.