search

wilsonngja / pe

0 stars 0 forks source link

A separate sequence diagram is not shown #12

Open wilsonngja opened 2 years ago

wilsonngja commented 2 years ago

image.png

A separate sequence diagram is not shown for the load from json file.

nus-se-bot commented 2 years ago

Team's Response

Hi,

Thank you for your bug report. A separate sequence diagram is shown in the following section (3.4.2 Loading CarparkList from JSON file) in the Developer Guide:

image.png

This can also be seen linked here in the Developer Guide, when the sequence diagram is explained:

image.png

Thus, the team would reject this bug report as the sequence diagram does exist and is properly referred to in the description of the sequence diagram.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: image.png

Based on the website, the convention for the separate diagram should be denoted by "sd" and contained within the box.

Because there was no "sd" box, it could be ambiguous. Moreover since the separate diagram is not directly below the referenced frame, it's inconvenient for the reader to scroll back and forth, hence this was being flagged.