albertarielw / pe

0 stars 0 forks source link

Unable to take name with / #4

Open albertarielw opened 2 years ago

albertarielw commented 2 years ago

Steps to reproduce: enter: add -c n/Ravi s/o Mikhail a/123

Expected: adds a client with name Ravi s/p Mikhail at address 123

Please refer to section "Use of symbols in input values" on https://nus-cs2103-ay2223s1.github.io/website/admin/tp-pe.html#tp-practical-exam-pe

Actual: Error message

Screenshot:

nus-se-script commented 2 years ago

Team's Response

Rejected because our error message clearly states that names can only be alphanumeric.

Legal names are not expected to be used as strictly as in healthcare or government applications.

Property agents can easily get around this without using this symbol (e.g. so)

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

[x] I disagree

Reason for disagreement: This is considered a bug as per CS2103T requirements as shown below:

You can check this link for clarification purposes https://nus-cs2103-ay2223s1.github.io/website/admin/tp-pe.html#pe-overview-2