search

Howlong11 / pe

0 stars 0 forks source link

No whitespace between t/ and other inputs for edit is not allowed #8

Open Howlong11 opened 2 months ago

Howlong11 commented 2 months ago

image.png

As not including any information to tags should remove all tags of the contact, if another input for the contact to be edited is followed without whitespace, it is better to accept it.

nus-pe-script commented 2 months ago

Team's Response

image.png

Firstly in our UG, the command format for edit has shown explicitly that there must be whitespaces between fields.

image.png

Secondly, we have already specified that the if the user wants to remove all tags, there must not be tags after it. However in the screenshot, the tester input "t/e/charlotte@gmail.com". In this case, the full "e/charlotte@gmail.com" is treated as the tag the tester is trying to input, which is the expected behavior of our app.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]