search

nus-cs2113-AY1920S2 / pe-dev-response

0 stars 0 forks source link

delete patient NRIC's Sequence Diagram's mistake #672

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.

It seems that in your code, you do not have a class named "logic", so here is it better to draw a more specific diagram which includes PatientParser.class and DeletePatientCommand.class

ACCF705B-8C1D-405F-941C-BFB89C626E14.png

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

itskesin commented 4 years ago

Team's Response

We followed the AB3 DG closely. image.png

Our team have stated that this diagram is to explain how each components interact with each other. image.png

Duplicate status (if any):

--