SherwynNg / pe

0 stars 0 forks source link

Wrong numbering for extension #22

Open SherwynNg opened 4 months ago

SherwynNg commented 4 months ago

Steps to reproduce:

  1. In the DG, under use cases
  2. For use case: add a contact

Expected: I think the numbering for duplicate contact added is wrong as i think it should happen after 1 and should be 1a or 1b instead of 3. If it is 3a, it means that Dook already added in the contact.

Screenshot 2024-04-19 at 5.27.41 PM.png

nus-se-script commented 4 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]

DG: incorrect step numbering in find use case

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.


The extension should not be 2a and 2b. Because 2a and 2b mean these steps happen after the step 2 ends. However, if the step 2 already ends, the app should have successfully searched and displayed the matching contacts.

Screenshot 2024-04-19 at 5.37.37 PM.png


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

Their Response to the 'Original' Bug

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

Accepted - uneven step numbering used in extension portion in DG

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: I do not think that this is a duplicate as it is referring to different use case. I agree that both bugs are related to wrong numbering in extensions but they are referring to different use case. Hence, I feel that they should not be a duplicate of each other.


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]