search

zixin448 / pe

0 stars 0 forks source link

Sequence diagram not updated #14

Open zixin448 opened 2 years ago

zixin448 commented 2 years ago

Page 10 sequence diagram is not updated, I think the execute command now takes in 3 other boolean values

Screenshot 2022-04-16 at 3.20.18 PM.png

soc-pe-bot commented 2 years ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Inconsistency between the textual explanation of implementation and 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.

Screenshot 2022-04-16 at 3.37.10 PM.png

Screenshot 2022-04-16 at 3.37.24 PM.png

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

Their Response to the 'Original' Bug

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

This is a typo in the diagram in the Developer Guide that does not affect the usage of the app. Hence downgrading 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]

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: The reason is given in the original bug. The severity level should be Low.