search

bernarduskrishna / pe

0 stars 0 forks source link

Overly complicated, repeated diagram in DG #11

Open bernarduskrishna opened 2 years ago

bernarduskrishna commented 2 years ago

image.png

While this diagram might be correct, 90% of what's inside is already illustrated at the delete 1 sequence diagram. Could have highlighted which part is different rather than putting everything inside. e.g. foundInNameOrTagsPredicate.

nus-pe-bot commented 2 years ago

Team's Response

Severity.veryLow Justification: Merely cosmetic as user can zoom into app to see details. Similar to other examples given in CS2103T lecture which are grouped as verylow severity

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]