nus-cs2113-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Unexplained 'C' in sequence diagram #16

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 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.


What is this big bold C in this sequence diagram (circled in red below) for the AddAppointmentCommand. It is not explained in the DG what this C means.

image.png


[original: nus-cs2113-AY1920S2/pe-interim#24]

itskesin commented 4 years ago

Team's Response

image.png

We follow closely to AB3 DG, the c represents the return parameter, should have reduce the font size when I was editing the sequence diagram, will take note.

The bug severitiy level is reduced to Very Low because we believe it is purely cosmetic and does not affect the user in understanding our design.

Duplicate status (if any):

--