search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

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

Open nus-pe-bot opened 1 month ago

nus-pe-bot commented 1 month 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.

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

ziiqii commented 4 weeks 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.

Duplicate status (if any):

--