yhtMinceraft1010X / pe

0 stars 0 forks source link

User Guide command does not work when copy and pasted #8

Open yhtMinceraft1010X opened 3 years ago

yhtMinceraft1010X commented 3 years ago

Command used:

add A7654321J n/Betsy Crowe f/ENG p/91119222 e/betsycrowe@example.com
a/212 Orchard Road, #18-08 s/unvaccinated m/nose lift surgery in 2012

From User Guide: image.png

Result: image.png

If copy and pasted, the a/ prefix gets fused with the input for e/ prefix, resulting in an invalid command format error. I believe that most new users would copy and paste the commands from the user guide to try out the product, thus the medium severity rating.

nus-pe-bot commented 3 years ago

Team's Response

No response provided.

The 'Original' Bug

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

Add command example does not work

Copying and pasting the 2nd example given in the Add section in the UG does not work on the app. (missing space between the email field and the address field)

image.png


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

Their Response to the 'Original' Bug

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

The reason why it does not work is because there is a no space between the email and address parameters. I.e. the 'a/' prefix is stuck to the email address that came before. The example command in the UG does come with a space.

Testers have been notified to watch out for potential issues when copying and pasting from the UG where spaces might not be copied

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]

Reason for disagreement: This may have been a formatting problem as a result of conversion to pdf, which may be beyond the scope of the module. Nevertheless, I believe that it is still a valid concern and should be recategorised as NotInScope.

Also, with regards to the response "Testers have been notified to watch out for potential issues when copying and pasting from the UG where spaces might not be copied", I did not see any mention of this in the User Guide and I do not think it is fair to expect that all testers will take note of this. It would be better to quote a reference for this particular statement to clear any uncertainties.


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]