ng-andre / pe

0 stars 0 forks source link

Common class in Architecture Diagram has a floating component #11

Open ng-andre opened 2 years ago

ng-andre commented 2 years ago

image.png

Common class is not connected to any other components of the application, which could mislead developers to think that it is an unused component of the application. According to the DG, it contains a "collection of classes used by multiple other components, such asexceptions and messages", which is not very clear, as these classes do interact with the main application or the user.

nus-pe-script commented 2 years ago

Team's Response

As an architecture diagram, it was designed with the primary purpose of providing an overview of the different components involved within the codebase of the program. The original intention was not to map out the exact interactions between each component in all use cases, but instead provide a general mapping. As such, the architecture diagram alone should not be viewed as a definite layout of how the application works.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]