search

rainish2000 / pe

0 stars 0 forks source link

large sequence diagram could be broken up #9

Open rainish2000 opened 2 years ago

rainish2000 commented 2 years ago

This diagram could be broken up using a reference frame as its font is currently much smaller than the rest of the document and difficult to read:

image.png

soc-pe-bot commented 2 years ago

Team's Response

Same diagram with additional details.

The 'Original' Bug

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

Delete Attendance sequence diagram page 22

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.

The sequence diagram is too small for normal readers to read. Reference Diagrams should be used here instead to make the interactions clearer.

image.png

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

Their Response to the 'Original' Bug

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

There exists a zoom function. Also, the flaw is purely cosmetic (font issues). Severity should be 'VeryLow'.

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]

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]