search

nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Empty model component #826

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

image.png

[original: nus-cs2103-AY2223S1/pe-interim#702] [original labels: severity.VeryLow type.DocumentationBug]

Benjamin-Sim commented 1 year ago

Team's Response

The tester did not explain what is wrong with having an empty Model component or how an empty Model component would hinder the reader.

In any case, our team's stance is the following: We decided to keep the Model component as an empty box since it is not the main focus of this diagram. Hence, it is not the purpose of this diagram to go into the details of how the Model component works. The Model component is simply meant to be viewed as a "black box" that the ListPersonsCommand depends on.

This technique of omitting details of less important components is encouraged by the teaching team and is regularly used in other diagrams in the AB3 DG.

Therefore, we reject this issue as we don't feel it is a problem at all.

Duplicate status (if any):

--