search

shardhrv / pe

0 stars 0 forks source link

Confusing usage of `t/...` to refer to both tags and pets #2

Open shardhrv opened 4 days ago

shardhrv commented 4 days ago

Description

In the link and unlink commands, in order to link owners to pets, I need to specify the owner with oN where N is the index of the owner to a pet t/pT where T is the index of the pet. However, in the add commands the t/... flag is used to add tags to the pet, which is confusing to users as the same flag can mean two separate things given a context.

Possible Solution

Consider removing the need for the t/... command entirely and follow the format that is given by oN to specify owner. In this way to refer to a pet you would only need to write pT, which would be much more consistent with the command usage.

Steps to reproduce (if any)

Relevant Screenshots

image.png

image.png

In this screen shot both the pPET and t/TAG format is being used.

nus-pe-bot commented 16 hours ago

Team's Response

This has been covered in our developer guide as a planned enhancement

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

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]