search

nus-cs2113-AY2122S1 / pe-dev-response

0 stars 0 forks source link

DG: Sequence Diagram for logic component #224

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.

I don't think you need to include ... as you should only show those that are in the sequence diagram.

image.png

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

pos0414 commented 2 years ago

Team's Response

We included … so that developers are not confused to think that nfm, bm, nldm, and ui are the only arguments of the execute method. Moreover, we have also confirmed with Prof Akshay before that it is valid to use … to hide not-so-important arguments, as long as it is mentioned that we will do so.

image.png

Duplicate status (if any):

--