ddanelynn / pe

0 stars 0 forks source link

Missing command instructions for unsortp #9

Open ddanelynn opened 3 years ago

ddanelynn commented 3 years ago

In DG, it was stated to follow instructions in UG for unsort. However, no instructions for unsortp was found.

Screenshot 2021-04-16 at 2.41.16 PM.png

nus-pe-bot commented 3 years ago

Team's Response

Duplicate of 2016.

The 'Original' Bug

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

Unsort 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.


The unsort command is not present in the User Guide, perhaps due to an oversight when implementing the command in the later stages of the project?

image.png

The above instruction in the DG also becomes invalid.


[original: nus-cs2103-AY2021S2/pe-interim#2016] [original labels: severity.High type.DocumentationBug]

Their Response to the 'Original' Bug

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

Thank you for the issue.

The command is not present in the current UG, but the DG also gives space for us to consider implementations for future iterations. So, not every feature or implementation discussed in the DG have to be part of the UG/product.

image.png

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 response

Team chose [response.NotInScope]

Reason for disagreement: This feature is not a proposed implementation for future features as this command is already working in the product. Hence, this is not an excuse for missing documentation for the feature.


:question: Issue severity

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

Reason for disagreement: I believe missing documentation for the whole feature should not be treated as veryLow.