danielonges / pe

0 stars 0 forks source link

'target' variable not specified in sequence diagram in DG #9

Open danielonges opened 3 years ago

danielonges commented 3 years ago

In this sequence diagram on page 5, the deletePatient(target) method call from :Logic to :Model introduces a target variable which has not been specified.

nus-pe-bot commented 3 years ago

Team's Response

As the developer guide is meant for a developer to read, we believe that it is clear that target refers to patient specified. In addition, in the actual code, the parameter is named as target as well.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

[ ] I disagree

Reason for disagreement: [replace this with your reason]