jonpwk / pe

0 stars 0 forks source link

Add Command Example 1 Error #5

Open jonpwk opened 2 months ago

jonpwk commented 2 months ago

The provided example for /add command in the user guide gives an error.

Given Command: /add John Doe; 91234567; johndoe@email.com; 12 Kent Ridge Dr; 2024; Finance

Expected Result: Successfully added John Doe

Actual Result:

image.png

Readers would expect the provided examples to work, but errors are thrown.

nus-se-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Add Command Example 2 Error

The provided example for /add command in the user guide gives an error.

Given Command: /add John Doe; 91234567; johndoe@email.com; 12 Kent Ridge Dr; 2024; Software Developer; Finance

Expected Result: Successfully added John Doe

Actual Result:

image.png


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

Their Response to the 'Original' Bug

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

This is correct behaviour as we believe you copied the command wrongly as there should be a space between ";" and "Software". We also mentioned in our user guide that:

Screenshot 2024-04-20 at 4.46.37 PM.png

"Parameters must be separated by a semicolon and space, if no PREFIX specified."

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.Rejected`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]