search

weishuangtan / pe

0 stars 0 forks source link

Sequence Diagram for AddFoodCommand can be shortened. #19

Open weishuangtan opened 3 years ago

weishuangtan commented 3 years ago

image.png

The current sequence diagram as shown is quite lengthy and decreases its resolution greatly. Perhaps the bars could be shortened? Also, the condition [if ActivityList is null] should be written on the right hand side of "opt".

nus-pe-bot commented 3 years ago

Team's Response

HI, thank you for raising this issue. I might have moved the condition message for opt down as I thought I might be in the way of the reader. I have check with my TA and he said that writing the condition message below opt is accepted.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]