search

Trav1sT / pe

0 stars 0 forks source link

Delete sequence diagram has missing element #13

Open Trav1sT opened 4 years ago

Trav1sT commented 4 years ago

I'm not quite sure since I can't view the source code. But should there be a cross below deleteContactCommand image.png

nus-se-bot commented 3 years ago

Team's Response

The severity has been changed to veryLow because it does not affet any usage for developers to interpret the implementation of this feature. This issue is rejected because this sequence diagram is following the standard sequence diagram in the AddressBook-3 Developer Guide.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Can I confirm but d is not in use anymore though? Yea, indeed it is based on AB-3 DG.