search

SherwynNg / pe

0 stars 0 forks source link

Contraints in UG #8

Open SherwynNg opened 6 months ago

SherwynNg commented 6 months ago

Steps to reproduce:

  1. Input: "add -n BoB Smith -e x@x.xx -a Tampines 693021 -tampines -t friendtoo"

Expected: I expected it to mention in the UG that the address should not contain -t or any prefix since it is hard for parsing.

Actual: It allowed me to add as a new contact and -t is recognised as a tag as "ampines"

Screenshot 2024-04-19 at 4.33.14 PM.png

nus-pe-bot commented 5 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]

Having -t in my address

Steps to reproduce:

  1. input: "add -n BoBbbb Smith -e x@x.xx -a Tampines 693021 -tampines -t friendtoo"

Expected: I expected -tampines to be part of my address.

Actual: The input got recognised as my tag instead as "ampines"

Screenshot 2024-04-19 at 4.31.13 PM.png

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

Their Response to the 'Original' Bug

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

Accepted - bug happened last minute since we ommitted a space from prefix -t to -t

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.FunctionalityBug`] Originally [`type.DocumentationBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.VeryLow`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]