search

JulietTeoh / pe

0 stars 0 forks source link

Too many low level details in find section #18

Open JulietTeoh opened 3 years ago

JulietTeoh commented 3 years ago

Talking about the CLI syntax, for example "n/ will contain 2 AppointmentNamePredicates , 1 checking for alex, and the other checking for john", is too low level and only becomes confusing for the reader.

Screenshot 2021-04-16 at 3.40.24 PM.png

nus-pe-bot commented 3 years ago

Team's Response

Dear tester, we need to use CLI commands here so we can have some names or values to better explain in detail how the command works. It would be awkward to introduce random magi numbers out of nowhere in the later parts of the DG.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]