luminousleek / pe

0 stars 0 forks source link

Incorrect notation for UML sequence diagram in section 4.2 and 4.4 #18

Open luminousleek opened 2 years ago

luminousleek commented 2 years ago

Bug: The objects in the sequence diagram should have a colon in front of the class name, but this is not the case for some objects (e.g. InteractionCommand, CommandResult)

Section 4.2: Screenshot 2021-11-12 at 5.26.15 PM.png

Section 4.4: Screenshot 2021-11-12 at 5.30.21 PM.png

nus-pe-bot commented 2 years ago

Team's Response

The first diagram bug is a duplicate which we have already accepted. The second diagram, we accept the bug but with a downgraded severity to VeryLow since it is a cosmetic bug.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

[x] I disagree

Reason for disagreement: This is not a cosmetic bug, but incorrect UML notation.