search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Lower level details #5659

Open nus-se-bot opened 2 years ago

nus-se-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

Architecture diagram contains too many lower level details.

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

s7u4rt99 commented 2 years ago

Team's Response

It is a sequence diagram which is supposed to display how the method works, which contains lower level details. We disagree that there are too many lower level details. This is already a reference diagram created by abstracting out from another sequence diagram. This diagram only contains one member of logic and shows how the command interacts with the model.

Also this is not an architecture diagram, it is a sequence diagram.

Duplicate status (if any):

--