search

wanjuin / pe

0 stars 0 forks source link

Insufficient UML diagrams. #18

Open wanjuin opened 1 year ago

wanjuin commented 1 year ago

Class diagrams could be added for each feature instead of just sequence diagram for add movie feature, remove movie feature and see detail feature. Addition of classdiagram would make it clearer in how classes are connected with one another.

nus-pe-script commented 1 year ago

Team's Response

No details provided by team.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: severity.Medium. Understanding that this flaw will not cause major problems to user / developer guide reading the DG, this could be labelled as a medium issue instead.

This should not be severity.veryLow, this is because severity.veryLow should only be used for cosmetic problem whereas this issue is not cosmetic related. Hence, this should not be severity.veryLow.

image.png