search

guohuang88 / pe

0 stars 0 forks source link

Delete Sequence Diagram in Developer Guide #10

Open guohuang88 opened 3 months ago

guohuang88 commented 3 months ago

It should have a destroy marker (X) after the activation bar of DeleteCommand.

image.png

nus-se-script commented 3 months ago

Team's Response

Destroy markers are not necessary in UML sequence diagrams. The destroy marker there was just to illustrate that DeleteCommandParser is a sub parser of AddressBookParser

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: To clarify, I am referring to DeleteCommand instead of DeleteCommandParser. Also, to add on on the bug report, since the destroy marker is added for the DeleteCommandParser, it should be added on DeleteCommand too for standardisation.