nshian / nus-cs2113-pe

0 stars 0 forks source link

Incorrect UML notation for object headers in sequence diagrams #15

Open nshian opened 1 year ago

nshian commented 1 year ago

As per the notation taught in this module, objects participating in a sequence diagram should be prefixed by a ":" e.g. ":Add" or ":Parser". The ":" seems to be ommitted.

image.png

nus-pe-bot commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

All objects should start with ":"

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.


Screenshot 2022-11-11 at 17.51.56.png

All objects like Duke, Parse, Add, ModuleList, UI should begin with a colon :

Same as all other sequential diagrams


[original: nus-cs2113-AY2223S1/pe-interim#891] [original labels: type.DocumentationBug severity.High]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

No details provided by team.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]