dendendenden04 / pe

0 stars 0 forks source link

Sequence diagram for storage has unintended cursor #19

Open dendendenden04 opened 1 year ago

dendendenden04 commented 1 year ago

image.png

When screenshotting the sequence diagram, you might have forgotten to move the cursor. It is not blocking anything important, but technically should not be part of a sequence diagram.

nus-se-script commented 1 year ago

Team's Response

Issue is related to cursor in diagram

The 'Original' Bug

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

Cursor covering sequence diagram in DG

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.


image.png

The cursor happens to cover an important part of the sequence diagram here, covering the method call.


[original: nus-cs2113-AY2223S2/pe-interim#1108] [original labels: severity.Low type.DocumentationBug]

Their Response to the 'Original' Bug

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

The cursor should not be in this diagram. However this is a cosmetic issue and it is easily fixable by redoing the screenshots for our affected UML diagrams.

Hence, our team lowered the severity to VeryLow and accepted this as a bug.

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]