search

Whysochong / pe

0 stars 0 forks source link

too much info in diagrams #29

Open Whysochong opened 1 year ago

Whysochong commented 1 year ago

image.png

way too many things going on in this diagram perhaps can be separated in different diagrams

soc-se-bot commented 1 year ago

Team's Response

Similar to the issue #4972.

The 'Original' Bug

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

Activity Diagram for DelStaff might be 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.

image.png

The activity diagram is overly complicated. It contains many branches that make the text small and hard to read, and also presents many conditions that are not extremely important, and could perhaps be omitted.

[original: nus-cs2103-AY2223S1/pe-interim#4825] [original labels: type.DocumentationBug severity.Low]

Their Response to the 'Original' Bug

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

We felt that all of the conditions are extremely important as it informs us of what is being considered and how it was implemented. The only issue is that the diagram is a bit too small. Since this is a purely cosmetic issue, we decided to lower the severity to very low.

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]