search

Howlong11 / pe

0 stars 0 forks source link

Editing with empty t/ followed by another edit to tag is not allowed #9

Open Howlong11 opened 2 months ago

Howlong11 commented 2 months ago

image.png

The app currently does not allow cases where the tag to be edited is followed. It would be better to let it delete all tags, then add the new tag specified

nus-pe-script commented 2 months ago

Team's Response

image.png

It is already specified in the UG that to clear a contact's tags, there should not be any specified tags after it. Thus this is the expected behavior of the app.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: the UG's description on 'without specifying any tags after it' is insufficient for the users whether it means there should be no tags at all after it or it means just to leave the 'tag to edit section' which should come after 't/', because multiple tags can be used for a single edit command.

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