nus-se-script
commented
1 year ago Team's Response
No details provided by team.
The 'Original' Bug
[The team marked this bug as a duplicate of the following bug]
Unable to take name with /
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:
[original: nus-cs2103-AY2223S1/pe-interim#3621] [original labels: severity.Medium type.FunctionalityBug]
Their Response to the 'Original' Bug
[This is the team's response to the above 'original' bug]
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 duplicate status
Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
:question: Issue response
Team chose [response.Rejected]
- [x] I disagree
Reason for disagreement: Although the error message state that alphanumeric characters are the only things allowed the teams should be aware that some names have special characters as well. Furthermore, real estate agents may want to include comments beside the name in brackets as well for easy differentiation.
Please refer to the section "Use of symbols in input values" on https://nus-cs2103-ay2223s1.github.io/website/admin/tp-pe.html#tp-practical-exam-pe
This section also states that this functionality is clearly a bug. The tester had awarded a very low as there was a workaround as stated by the dev team which would make this a cosmetic issue. However, since it was marked as a duplicate this tester will be going along with the original bug and its severity level as well.
:question: Issue severity
Team chose [severity.Medium]
Originally [severity.VeryLow]
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
Description: Adding names with slashes in them causes the system to not take in the name.
Steps to reproduce:
Image of error:
Reason for Bug level: As real estate agents would like to store their clients full name to prevent confusion as they might have a lot of clients. However this can be overcome by writing son of instead of s/o etc. that is why it has been given a very low level.