search

JensonWee / pe

0 stars 0 forks source link

Error in Sequece Diagram example in DeveloperGuide #8

Open JensonWee opened 4 years ago

JensonWee commented 4 years ago

Description: Sequence Diagram is created wrongly. Example: HAMS:Duke class creates ui:UI but there is no arrow back to it, same as parser:Parser. Not only that, the parser:Parser class as illustrated, exists after return new editPatientCommand, which should not be the case.

image.png

nus-pe-bot commented 4 years ago

Team's Response

I suppose the activation bar for parser should not be that long. And the return arrow to notify the user that patient records are not needed under minimal conditions.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]