nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Unmark command flag is inconsistent #127

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

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.


UG header states that the unmark command is supposed to be -m, but the format specifies that the unmark command is supposed to be -u. Also, the example given states that it should be -m.

UG Header:

image.png

Format:

image.png

Example given:

image.png


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

crvstalphua commented 1 year ago

Team's Response

Thank you for the bug report!

There are two errors brought up here:

Here we will mainly be addressing the first error, in line with the guidelines below:

Screen Shot 2022-11-14 at 7.26.39 PM.png

We acknowledge that we did miss out the -u flag for unmark command in the command section header, however, since the command is correctly stated in the command format as well as the command summary (which contains the full list of commands), we believe that this error would only provide a slight inconvenience to the user as they would likely have read through the entire section before using the command. We believe the severity should therefore be "Low" instead of "High" as most users would read the entire section prior to use, leaving only a few users confused that can easily refer back to the paragraph written below, and also in the worst case users can still use the application as they can simply delete the issue or re-add the issue to mark it as incomplete; it therefore does not qualify for high. Although it is not the main issue we are addressing, the same can also be said with regards to the second issue, which is a typo in the unmark command example.

Screenshot 2022-11-14 at 4.53.05 PM.png

Screenshot 2022-11-14 at 4.53.16 PM.png

Duplicate status (if any):

--