search

qinxutan / pe

0 stars 0 forks source link

Delete command prerequisite wrong #7

Open qinxutan opened 2 months ago

qinxutan commented 2 months ago

Same as the find command issue:

Expected: when i find a person -> return someone -> delete person

C888D794-38B4-43E8-B00B-B9A02C87BC18.jpeg

Actual: no person returned since i am not in a class view

Steps to reproduce: edit the delete command to: Prerequisites: At least one person shown in the interface. You may select a class, and then find a person or add a person or use list to see all persons in a class

nus-se-script commented 2 months ago

Team's Response

Hi, we understand why you may have misinterpreted this. The only prerequisite is that 'at least one person shown in the interface', the rest are just suggested commands to accomplish this.

Under the find, add and list command instructions, the prerequisite to select a class is already stated. The reason why it is phrased that way is that select as an individual command will list people.

We feel that this would not cause any confusion to users

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]