search

rageqqq / pe

0 stars 0 forks source link

Missing class diagrams for UI and Parser components #16

Open rageqqq opened 3 years ago

rageqqq commented 3 years ago

image.png

image.png

image.png

image.png

The UI, Editor and Parser components do not have a class diagram to describe their architecture while the Model, Storage and Command components have class diagrams. Might be useful to have some form of consistency in providing diagrams for components

nus-pe-bot commented 3 years ago

Team's Response

We believe there is no requirement to put a class diagram for every component. In this case, we deemed it unnecessary as both components only contain a single class each. The user can refer to the architecture diagram to find out how these 2 components interact with the other components.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Editor component has 2 classes? Why not create a diagram for that includes both text editor and command components