search

sagarsureka / pe

0 stars 0 forks source link

Error of UML diagram #13

Open sagarsureka opened 3 years ago

sagarsureka commented 3 years ago

Missing merge node of "currentDay > today - 7" and the conditions with the branch nodes are not in square brackets Screenshot 2020-11-13 at 2.58.36 PM.png

nus-pe-bot commented 3 years ago

Team's Response

I am trying to show a while loop. I definitely agree that there is a better way of doing it, such as stating it as a do-while loop. But the lack of a merging node is because it does not make sense for to have a merging node.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: No reason giving for downgrading of the bug. Maybe the severity should be Medium as it does cause incovenience for people trying to interpret the diagram as the lack of a merging node is confusing. But the reader can still try to make sense of it because of evidence of it being a loop.