DesmondTo / pe

0 stars 0 forks source link

Add feature behavior does not match that of mentioned in the user guide #6

Open DesmondTo opened 3 years ago

DesmondTo commented 3 years ago

Step to reproduce:

  1. Enter command add n/Joe s/fulltime r/kitchen p/98765432 $/1234789 e/Joe@example.com

Expected:

image.png

The staff with the specifying detail is added

Actual: Not added. Error message displayed.

image.png

nus-pe-bot commented 3 years ago

Team's Response

Error is because the salary cannot exceed 9999.99. It is a documentation bug using an invalid example.

The 'Original' Bug

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

Incorrect example in UG

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.


Salary is over the limit.

image.png

image.png


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

Their Response to the 'Original' Bug

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

Downgraded as it was a typo in the example. Valid salary inputs are specified in the error message (provided image) as well as in the Flag Legend in the User Guide.

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 type

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: image.png

Based on the CS2103T website, if the behavior is different from UG, although it is because UG is outdated, still consider as functionality bug.

image.png Also, as you can see here, outdated UG is not count as a documentation bug.


:question: Issue severity

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

Reason for disagreement: image.png

This situation is similar to this question in PE readiness quiz. Should be medium.