search

Gavzzz / pe

0 stars 0 forks source link

Deleting range not specified #9

Open Gavzzz opened 1 year ago

Gavzzz commented 1 year ago

When deleting a patient the range of numbers to input is not specified as the UG assumes the user knows to input the smaller number followed by the larger value and when inputting a larger value followed by a smaller value it returns an error

image.png

image.png

nus-pe-script commented 1 year ago

Team's Response

image.png.

The examples given illustrate clearly that delete 1 3 will delete patients from index 1 to 3 inclusive. Hence, more details about the indices are omitted. In addition, there is also feedback from the app if the index that the user keys in is not valid.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Only correct inputs were given in the user guide and error message also does not tell the user why the deleting range is invalid as it only tells the user that it is wrong. Hence User needs to assume that the only correct deleting range must be from a smaller index to a larger index