Tan-Jin-Waye / pe

0 stars 0 forks source link

Implementation of Edit Internship: Inconsistent return arrow & activation bar. #8

Open Tan-Jin-Waye opened 1 year ago

Tan-Jin-Waye commented 1 year ago

The sequence diagram specified does not have return arrows and activation bar for the getFilteredInternshipList(), which should be reflected as the returned ObservableList should not be abstracted out, since it is used by the function.

Screenshot 2022-11-11 173524.png

nus-pe-script commented 1 year ago

Team's Response

The ObservableList was omitted as it is a low-level detail in the Model component, while the focus of this sequence diagram is to showcase the Logic component.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]