search

kester-ng / pe

0 stars 0 forks source link

UG bug: Perhaps state that you need minimal number of items, as the example command, the index given was quite big #9

Open kester-ng opened 4 years ago

kester-ng commented 4 years ago

The UG example given was very big, not great for users to try out the commands. A smaller index would be better.

Screenshot 2020-04-17 at 3.09.08 PM.png

nus-pe-bot commented 4 years ago

Team's Response

The commands given were only examples, and should not be taken to be correct in all cases. Obviously the index will depend on the size of the customer/ product list.

Moreover this is already included in the DG. Under instructions for manual testing: 'Other incorrect add commands to try: addt c/x p/1 q/2 (where x is larger than size of customer list)' So from here, you can also see that the customer/ product index cannot be above the size of customer/ product list.

If we were to mention that at least x number of customer/ product must be in the list before the following examples will work, we will have to write it for all examples in the UG, which will make the UG even longer and crowded. Thus it is unnecessary info that can be inferred easily and it is not a bug.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]