search

maanyos / pe

0 stars 0 forks source link

floating class in Logic diagram #10

Open maanyos opened 2 years ago

maanyos commented 2 years ago

how does XYZCommand class relate to the logic component? is it different from the ABCCOmmand below?

image.png

nus-se-script commented 2 years ago

Team's Response

Hi, thank you for your response.

The 'Original' Bug

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

Unexplained XYZ command in the Logic component

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.

There's a XYZ command class in the Logic diagram without any connection to any other component.

The documentation also did not mention XYZ command at all in the section or the section prior. Also it causes confusion with the ABC command further down the diagram.

image.png

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

Their Response to the 'Original' Bug

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

Hi, we will be accepting this bug. It seems that we forgot to delete the extra class.

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]