search

Jillzyt / pe

0 stars 0 forks source link

[UG][Delete Supplier Command] Index information #18

Open Jillzyt opened 3 years ago

Jillzyt commented 3 years ago

Index message may be clearer to state from 1 to the length of the list as zero may seem valid to some users. image.png

nus-se-bot commented 3 years ago

Team's Response

Tester screenshot is incomplete, "from 1 to the length of the list" is included in the second half of the delete command in the UG.

In the case where users do not know that positive integer does not include zero, index zero is not used in our GUI and appropriate error message will be shown to users whenever there is an attempt to use index zero.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The team was unclear on where to find the from 1 to the length of the list is included in the second half of the delete command in the UG.

Tried to ctrl+F to find the text. However, could not find such.

image.png

The screenshot below is the extended picture of delete section. image.png

:question: Issue severity

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

Reason for disagreement: The bug is not only cosmetic and affects the usage of the application at rare occasions as the user may expect a different output.