skythefire / pe

0 stars 0 forks source link

Mismatch command format and examples for `deletefromorg` in UG #4

Open skythefire opened 2 years ago

skythefire commented 2 years ago

The given command format in the UG is: It does not include the prefix n/ for name.

image.png

However, the examples given in the UG do not correspond to the given command format.

image.png

nus-pe-bot commented 2 years ago

Team's Response

This bug is noted before. This is a duplicate because "n/" has been noted before as a typo in UG.

The 'Original' Bug

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

Incorrect command format given for deletefromorg

The user guide states that the command format for deletefromorg is:

image.png

Test input: deletefromorg 1 Shopee

Expected input: Contact Alex Yeoh deleted from Shopee

Actual input:

image.png

Invalid command format error received. The Dev team likely missed out the n/ prefix in their user guide command format.


[original: nus-cs2103-AY2122S1/pe-interim#5250] [original labels: severity.Low type.FunctionalityBug]

Their Response to the 'Original' Bug

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

Thank you for pointing this out. Downgraded severity: This seems to be a typo because in all the examples we clearly mention the "n/" input format.

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 missing n/ affects the usage of the deletefromorg command, as users may not be able to utilise the command, due to the incorrect command format given in the User Guide. Yes, it is a typo but it is a typo that affects the usage of the command and is not purely cosmetic.

image.png