munharsha / pe

0 stars 0 forks source link

DG bug under logic component #19

Open munharsha opened 4 years ago

munharsha commented 4 years ago

Since you are only using one parser for this project, the diagram should use TaskmasterParser instead of XYZCommandParser. This is so as the namings do not match aslo. It should be TaskmasterCommandParser instead if you want to use XYZCommandParser Screenshot 2020-11-13 at 1.25.10 PM.png

nus-pe-bot commented 4 years ago

Team's Response

This has been accepted as a documentation bug. However, this bug was in a relatively insignificant part of the DG (discussing about an alternative design vs being part of the actual design of the code).

Items for the Tester to Verify

:question: Issue severity

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

[ ] I disagree

Reason for disagreement: [replace this with your explanation]