search

yapdianhao / pe

0 stars 0 forks source link

SEquence diagram too tiny #11

Open yapdianhao opened 4 years ago

yapdianhao commented 4 years ago

The sequence diagram seems to squeeze everything in it. Consider reference frames to clearer images

Screenshot 2020-04-17 at 3.23.35 PM.jpg

nus-pe-bot commented 4 years ago

Team's Response

Thank you for the suggestion, our team decides that since the sequence diagram can be read, severity is lowered from Low to VeryLow. Furthermore, the diagrams can be zoomed in if needed.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Sequence diagrams have the reference frame functionality to break down complicated sequence diagrams and omit unnecessary details, for example in the diagram, get commandResult, or the test method. The words in the diagram is clearly too small, especially developers will have a hard time looking at long, squeezy words such as the function name above. A reference frame clearly is essential here. Furthermore, asking developers to zoom in, then zoom out while digesting materials in the DG will be extremely painful.

:question: Issue severity

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

Reason for disagreement: This bug actually 'causes inconvenience to some users but they are still able to use the product'. This is a bug level of Medium according to the module website, certainly not a negligible issue like a typo to be downgraded to a VeryLow severity.