search

nus-cs2113-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Lack of Activation bars in sequence diagrams #308

Open nus-pe-bot opened 6 days ago

nus-pe-bot commented 6 days ago

Screenshot 2024-11-15 at 5.27.44 PM.png

[original: nus-cs2113-AY2425S1/pe-interim#65] [original labels: severity.Medium type.DocumentationBug]

Progresst-8 commented 5 days ago

Team's Response

The severity of this should be very low, as the lack of activation bars in sequence diagrams is a purely cosmetic flaw and does not affect the usage of the program

Moreover, the CS2113 website states that activation bars can be omitted.

image.png

As stated in the DG screenshot you posted, this class primarily delegates its functionality by calling methods from other category classes. The purpose of the sequence diagram still conveys which category classes CategoryFacade calls for each of its key methods. More precise details like activation bars are not required to convey this idea.

Thus, this is not a bug.

Duplicate status (if any):

--