ruishanteo / pe

0 stars 0 forks source link

Sequence diagram for shift day is missing activation bar #22

Open ruishanteo opened 9 months ago

ruishanteo commented 9 months ago

Screenshot 2023-11-17 at 5.24.43 PM.png

On the right, getShiftDays() method is missing an activation bar.

soc-pe-bot commented 9 months ago

Team's Response

Hi tester! As stated in the textbook, activation bars can be omitted as long as it does not introduce any ambiguities, which it does not in this case.

Screenshot 2023-11-18 at 11.55.00 PM.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I do not see why you would omit the inclusion of this one activation bar, while including others, like the one at :Model (which can be argued to be unambiguous as well).

The level of inclusion of details is inconsistent throughout the diagram.


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]