SeanHoWB / pe

0 stars 0 forks source link

Cannot see sequence diagrams #18

Open SeanHoWB opened 2 years ago

SeanHoWB commented 2 years ago

image.png

Image not shown, tried on : adobe Acrobat Reader DC microsoft edge firefox

nus-pe-bot commented 2 years ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Missing sequence diagram from view order

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


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

Their Response to the 'Original' Bug

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

Thanks for reporting. We have a ViewOrder.puml file which can be found in the docs section in our github. When we used the plantuml proxy to see this file which we used in the DG, it gives a website error which is not under our control. Hence, we are lowering this to low severity.

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.Low] Originally [severity.Medium]

Reason for disagreement: This bug is totally within your control. You could have saved the image of the diagram into your tp under an image folder to avoid using an external proxy. Furthermore you could have checked the UG upon submission, and during the nbox upload to see if it was loaded correctly. Only through gross neglience were you able to have the diagrams not shown in the DG.

image.png

As a reader, the lack of diagrams here made me unable to understand how the view order feature work, and hence is a medium severity bug