search

wwweert123 / pe

0 stars 0 forks source link

Possible incorrect naming of class in sequence diagram #15

Open wwweert123 opened 12 months ago

wwweert123 commented 12 months ago

image.png I am not sure if I misunderstood, Shouldnt the ":Main" be ":CafeCtrl" as shown in the previous sequence diagram

soc-se-bot commented 11 months ago

Team's Response

Thanks for pointing that out, it is indeed a typo and the name of the class should be CafeCtrl. However, since CafeCtrl is the main class of the app and it contains main method, I feel that using Main wouldnt significantly hinder the user from understanding how Parser component interacts with other component. Hence, perhaps the severity is lower than a medium?

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]