search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

UML sequence diagram for ViewCommand #4195

Open nus-pe-bot opened 2 months ago

nus-pe-bot commented 2 months 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

Description

The existence of the Model component confuses the reader.

Placing the model component there will give the incorrect impression that model component is initialized during that period for no use.

Idea

If the component was not used during the interaction, it should be removed from the diagram.

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

hiivan commented 2 months ago

Team's Response

Thank you for the feedback!

Duplicate status (if any):

--