juliusgambe / pe

0 stars 0 forks source link

[DG] Missing Diagram #18

Open juliusgambe opened 9 months ago

juliusgambe commented 9 months ago

Cannot see diagram Screenshot 2023-11-17 at 5.40.02 PM.png

nus-pe-script commented 9 months ago

Team's Response

Thank you for pointing this out! However, it does not have much impact on the understanding of the DG since the sequence diagram is just supplementing what is already explained in words. Thus, the severity should be Low

The 'Original' Bug

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

Sequence diagram for "Deleting Time from a Person" does not load

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

Under DG -> Implementation -> Deleting Time from a Person

The sequence diagram for deleting time from a person does not load.


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

Their Response to the 'Original' Bug

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

Thank you for pointing this out!

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.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]