nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

missing colon : for sequence diagram #453

Open nus-pe-bot opened 6 months ago

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

Description of bug

missing colon, should be :Grocery or :Storage sequence diagram at page 18 of developer guide

Screenshots

_{[original: nus-cs2113-AY2324S2/pe-interim#517] [original labels: severity.Medium type.DocumentationBug]}

64-1 commented 5 months ago

Team's Response

The labels at the top of the lifelines are meant to represent the classes involved in the sequence rather than specific instances. so i don't think needs to add colon.

Duplicate status (if any):

Duplicate of #2309