nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

addcommand in delete sequence diagram #4925

Open nus-se-script opened 4 months ago

nus-se-script commented 4 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.

why is there an addcommand when this sequence diagram is for delete 1

_{[original: nus-cs2103-AY2324S2/pe-interim#4465] [original labels: severity.Low type.DocumentationBug]}

bachletuankhai commented 4 months ago

Team's Response

The issue does not point out any problems with the diagram mentioned.

Answer for the question: the execute method of DeleteCommand does call model.addCommand.

nus-cs2103-AY2324S2 / pe-dev-response

addcommand in delete sequence diagram #4925

Team's Response

Duplicate status (if any):