keke101 / pe

0 stars 0 forks source link

Extra Parameters Allowed #10

Open keke101 opened 3 years ago

keke101 commented 3 years ago

Parameters that are not required will still be accepted as a valid command.

E.g. random bunny 123123 image.png

nus-pe-bot commented 3 years ago

Team's Response

Rejected as it is by design for the app to accept commands with typos as long as the key components required by the command are present. The reason is to prevent frustration of the user when they make typos while fast typing.

Again, by typing the key component of the command and having spelled it correctly shows intent that the user wanted to use the command (it is highly unlikely for the user to keysmash the command by accident), especially for simple commands like list bunny with no argument. Thus it makes sense for the app to proceed with carrying out the comand despite the typo of adding extra characters.

The app is still usable despite accepting additonal/ excessively typed commands, and can still perform the necessary commands as expected.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]


:question: Issue severity

Team chose [severity.Low] Originally [severity.VeryLow]

Reason for disagreement: [replace this with your reason]