marcus-ny / pe

0 stars 0 forks source link

Developer Guide's UI component has missing UI components (for Display List Panel and the Schedule Panel) #13

Open marcus-ny opened 4 months ago

marcus-ny commented 4 months ago

Description: On the app's GUI, there is a person list panel (which is AB3 default) and the new Schedule Panel that is displayed side by side, as well as the display list panel (which is described in a later part of the DG). However, this implementation is not updated in the DG's UI component. This may cause some confusion over how these display and schedule panels integrates with the rest of the GUI components and the app itself.

Screenshot 2024-04-19 at 5.16.47 PM.png

nus-pe-script commented 4 months ago

Team's Response

The DG did not incorporate Display List Panel and the Schedule Panel. 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.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]