search

lordgareth10 / pe

0 stars 0 forks source link

DG: Implementation section 11 diagram is too complicated #23

Open lordgareth10 opened 4 months ago

lordgareth10 commented 4 months ago

Screenshot 2024-04-19 at 5.43.23 PM.png Perhaps it would be better to show the diagram which does not include all the possible exception calls as they do not add value to a readers comprehension. These added details make the rest of the diagrams text too small to read without zooming in.

soc-se-bot commented 4 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

[DG]: diagrams are too complicated

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.

You can't even see it clearly, and for most of the diagrams, they have the same problem, too complicated.

e.g.1 image.png

e.g.2 image.png

[original: nus-cs2113-AY2324S2/pe-interim#2080] [original labels: severity.Medium type.DocumentationBug]

Their Response to the 'Original' Bug

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

For the first diagram, 64-1 accepted the issue, sorry. For 2nd diagram, we felt that it is not consider as long or complicated.

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]