search

youdonnnn / pe

0 stars 0 forks source link

Message box displays incoherent message when using list command #7

Open youdonnnn opened 6 months ago

youdonnnn commented 6 months ago

I tried to list all persons in "fin" which listed all persons with the finance tag but the message box now says "Listed all persons in fin" which does not make sense

image.png

nus-se-bot commented 6 months ago

Team's Response

Hi, we are unable to view your image.

We believe this more of a feature flaw as it seems to be not complete, not that the functionality is not working.

Also, if you type out any full tag, such as finance, the message makes sense. If it makes sense to find all the people in a group(tag) that have the word fin which can be short for finance, then the message also makes sense. There should be no issue if you follow this logic.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Maybe the message should be changed to something that is more general? Right now I could try ls alic and the message shown would be "Listed all persons in
alic." Having the message just be "Listed all persons." might be better.

This could be a small issue your team fixes in the future!

Also sorry for the picture, not sure what happened there.

## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]