search

YC-Michael / pe

0 stars 0 forks source link

Method call not consistent between Sequence diagram and explanation #13

Open YC-Michael opened 1 year ago

YC-Michael commented 1 year ago

image.png

As quoted "In the command line, the user inputs find MaLaXiangGuo to find a recipe from the recipe list. Duke calls the parseCommands() method in the Parser class", the arrow from user should point directly to Duke class instead of Ui class.

nus-pe-bot commented 1 year ago

Team's Response

The arrow shown is correct as we implemented it as what is shown on the sequence diagram, however we understand that the description is lacking in the explanation for the UI component and hence we would take this as a Low severity.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]