search

jovantanyk / pe

0 stars 0 forks source link

[DG] JsonAdaptedEvent not shown in UML Diagram #16

Open jovantanyk opened 2 months ago

jovantanyk commented 2 months ago

JsonAdaptedEvent has an association with JsonSerializableCalender as each JsonSerializableCalender class contains a list of JsonAdaptedEvent. This should be reflected in the UML Diagram Source Code: image.png

Actual UML Diagram in DG: image.png

nus-se-script commented 2 months ago

Team's Response

Good catch.

The 'Original' Bug

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

[DG] DisplayCard not shown in DG UML Diagram

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.

DisplayCard class not represented in the DG UML Diagram despite extending UiPart and having an association with DisplayListPanel

In Source Code: DisplayCard extending UiPart: image.png DisplayCard having an association with DisplayListPanel: image.png

In DG:

image.png

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

Their Response to the 'Original' Bug

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

Good catch! We changed the severity from very low to low as this could be inconvenient for developers.

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