search

jeffsieu / pe

0 stars 0 forks source link

Diagrams are too complicated #15

Open jeffsieu opened 2 years ago

jeffsieu commented 2 years ago

Under batch import/exports, the diagrams are too complicated. A bulk of the diagrams are the parsing logic which was already covered in one of the diagrams above in the DG.

image.png

image.png

nus-pe-bot commented 2 years ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Sequence diagram too small too read when zoom to fit width on adobe acrobat reader

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

What is written:

image.png

When I opened this on my pdf reader, the default settings is zoom to fit width (same for majority I believe) and the contents were arguably small to read unless it is zoomed in a few times. In addition, not all the contents can be fit into the screen when I do so.

As such, I feel that this could be an example of too much low level detail fit into the sequence diagram.

[original: nus-cs2103-AY2122S1/pe-interim#2493] [original labels: severity.VeryLow type.DocumentationBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Accepted. We agree that image is too small.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]