wuzhzn / pe

0 stars 0 forks source link

Additional fields accepted after command #6

Open wuzhzn opened 1 year ago

wuzhzn commented 1 year ago

image.png

This also works for clear clear, bye bye and other similar commands

soc-se-bot commented 1 year ago

Team's Response

We are rejecting this on the basis that the feature is able to work properly without causing any inconvenience to the user, and this is an enhancement to the flexibility of our commands to allow any typos after actual command or if the user is actually more inclined to typing "bye bye" rather than "bye".

Attached below is how a feature flaw is defined in the module. It would be helpful for us if you could give a little more reasoning on why you have flagged this as a bug thank you!

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: It is less useful since there are commands that are different from the examples I stated above. For example, clear clear, bye bye accept additional fields, whereas there are commands that do not accept additional fields such as recipe , or make , or add /qty . The difference in commands might confuse fast typists or those who want to use these features fast if they use the command that allows additional fields and then use the command that does not allow additional fields. It would be much easier for users if all the command either accept or do not accept additional fields.