NiniJiaying / pe

0 stars 0 forks source link

[DG] Logic Component graph #7

Open NiniJiaying opened 3 years ago

NiniJiaying commented 3 years ago

It was specified that "the CommandResult also contains a UiCommand object". However, in the graph, the "container-containee" relationship is not reflected at all.

nus-pe-bot commented 3 years ago

Team's Response

Screenshot 2021-04-17 at 4.15.32 PM.png

As seen in the screenshot above, the association is clearly shown. Aggregation is not used in line with module recommendations to avoid aggregations as they add more confusion than clarity. From the module website,

Screenshot 2021-04-17 at 4.19.42 PM.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]