search

ziyingli / pe

0 stars 0 forks source link

Display command sequence diagram phase 2 #15

Open ziyingli opened 4 years ago

ziyingli commented 4 years ago

Abit strange to put both UI components MainWindow and StatisticsWindow separately. MainWindow should be boxed under UI

Screenshot 2020-04-17 at 3.32.12 PM.png

nus-pe-bot commented 4 years ago

Team's Response

noted. will edit the sequence diagram for better clarity. however, this issue is purely cosmetic and will not affect another developer's understanding of the diagram so we made the severity very low.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your reason]