search

jwdavis0200 / pe

0 stars 0 forks source link

Commons class is used by multiple classes but yet is not so in the architecture diagram. #12

Open jwdavis0200 opened 1 year ago

jwdavis0200 commented 1 year ago

image.png

The commons components is shown to be isolated here, which may confuse developers and give them the impression that it is either a deprecated component or an isolated component in the architecture.

However, later in the DG,

image.png

It is specified that the commons class is used by multiple components. Hence not accurate diagram.

soc-se-bot commented 1 year ago

Team's Response

Commons package is placed inside the box with other packages which shows the main folders used for the application.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I understand that the box is with the other packages yes, but the other classes have navigability notations between them accordingly while commons class, which has navigability with the other packages as well, is not shown here.