search

NgYaoDong / pe

1 stars 0 forks source link

Bug for redundant class in class diagram #11

Open NgYaoDong opened 5 months ago

NgYaoDong commented 5 months ago

Documentation Bug (Severity LOW)

Would the CantVasMain class still be necessary since readers already know that that class is the main class and will always be used?

image.png

nus-pe-script commented 5 months ago

Team's Response

It is true that CantVasMain is not necessary for the sequence diagram, but it does not affect the clarity and flow of logic of the sequence diagram as a whole. Therefore, it will be more fitting for the severity of this issue to be very low instead of low.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]