t1mzzz / pe

0 stars 0 forks source link

Example `add` command displayed not correct #6

Open t1mzzz opened 1 year ago

t1mzzz commented 1 year ago

Procedure:

  1. Run help add, which displays the example add command (seen below)

image.png

When abiding by the example add command seen above (i.e. add n/Abc p/1234 e/a@gmail.com a/PGPR s/0A in progress st/Java dt/Bachelors jt/Internship rate/4), there is an error as seen below.

image.png

This is because of the jt which based on the User Guide should be jtt.

Minor error but if the user does not view the UG they would have no clue how to add a job type tag.

nus-pe-script commented 1 year ago

Team's Response

We feel that this is an valid bug, but the issue has been reported wrongly. The help command gives the proper 'jtt' flag, unless we are mistaking the syntax/error reported by the person.

Instead, perhaps what the tester is referring to is the add command documentation where the example command given gave the wrong jt flag instead of jtt:

image.png

We have accepted this issue as a duplicate of a previous issue, as they both refer to the same incorrect example given in the add command.

The 'Original' Bug

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

Example for Add is Wrong

image.png

The job tag for John Doe should be "jtt" instead of "jt". Considered a minor documentation error.


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

Their Response to the 'Original' Bug

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

We agree that this is a valid bug. However we feel that the severity should be Very Low since it's a typo, as we have clearly mentioned the correct format in the command format, as shown below.

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 severity

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

Reason for disagreement: [replace this with your explanation]