search

gremmyz / pe

0 stars 0 forks source link

Invalid command format feedback message #5

Open gremmyz opened 1 year ago

gremmyz commented 1 year ago

Type: Design Flaw

Currently,

when addPerson is used, it returns an invalid command format message. The example command in the command format message is as follows:

Example: addPerson n/John Doe p/98765432 e/johnd@example.com t/friends t/owesMoney

image.png

Perhaps the example command should be updated such that the tags suit the product more, such as:

Example: addPerson n/John Doe p/98765432 e/johnd@example.com t/researcher t/analyst

This would be similar to the default data that is given in the application.

soc-pe-bot commented 1 year ago

Team's Response

The tags given in the error message is just an example format for the librarian to follow. Although it could be more relevant to what we are trying to portray, it does not affect the functionality of ExpressLibrary in any way.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: > it does not affect the functionality of ExpressLibrary in any way.

Yes, I agree that it does not affect the functionality, and hence it is not a functionality bug.

However, it is a feature flaw. It is not representative of your target user group.

image.png

The reason for the severity.VeryLow is because, as you mentioned, it does not affect functionality, it is purely cosmetic. I believe this should be either response.Accepted, or response.notInScope if the team had other things to do with the time they had.