search

nus-cs2113-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Error in Sequece Diagram example in DeveloperGuide #258

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

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

[original: nus-cs2113-AY1920S2/pe-interim#257]

yukilite 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.

Duplicate status (if any):

--