Austintjh19 / pe

0 stars 0 forks source link

Invalid Prefix allowed for Commands. (e.g. find n/Google rand/Austin) #11

Open Austintjh19 opened 1 month ago

Austintjh19 commented 1 month ago

image.png

image.png

nus-pe-script commented 3 weeks ago

Team's Response

image.png

In the DG, we are planning to have names that are of non-alphanumeric characters, so it is possible that some companies might have names that contain special characters like /. In this case, this function behavior is justified.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: How about finding by Funding stage for instance ?

find f/A rand/Austin executes despite rand/Austin clearly not being a funding stage parameter.

image.png

Same with edit-p command:

edit-p 1 1 rand/Austin pn/John pe/johndoe@example.com executes.

image.png