search

ThatGirlJam / pe

0 stars 0 forks source link

DG wrong number in diagram #29

Open ThatGirlJam opened 6 months ago

ThatGirlJam commented 6 months ago

Says cancel 19 but diagram is cancel 1

Screenshot 2024-04-19 at 17.37.23.png

nus-se-script commented 6 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Sequence diagram does not match given command

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.

As seen below, the example given for the remove orders feature is cancel 19, but the sequence diagram shows what happens for cancel 1 instead.

image.png

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

Their Response to the 'Original' Bug

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

Downgraded severity to very low as it is a minor typo in '19'.

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