AlexanderTanJunAn / pe

0 stars 0 forks source link

Expanding Find function #7

Open AlexanderTanJunAn opened 3 years ago

AlexanderTanJunAn commented 3 years ago

As the program is meant for event management, it is not ideal to always be asking for the last 4 digits of a person's ID. Maybe can expand find to work on both n/ and p/.

nus-pe-bot commented 3 years ago

Team's Response

The purpose of the find command is to obtain the details of one specific visitor. However, when we use name or phone number to find a person, it may return a list of visitors as these are not unique fields. Thus, it will defeat the purpose of the find command.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I disagree as personnel of event management might not always have the ID digits to work with, one example of this is if Friend A is enquiring if Friend B has checked in, i highly doubt Friend A would know Friend B's ID's. Hence, it would be better if it checks through all fields.


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]