ricketytoc / pe

0 stars 0 forks source link

Cannot add student with name containing '/' #3

Open ricketytoc opened 10 months ago

ricketytoc commented 10 months ago

Description

When I try to add a name that contains '/' the app says that Names should only contain alphanumeric characters.

Expected result Allow the addition of student with names with special characters.

Actual result Does not allow.

Justification There are students whose names contain 's/o'. I think that not allowing names with special characters might be an overzealous restriction. I think that it is medium severity because it might cause inconvenience to tutors as they might get confused about the actual name of their tutees as they need to substitute the special character with a sequence of special alpha numeric characters.

soc-pe-bot commented 10 months ago

Team's Response

Issue accepted however, we believe the severity is not medium but instead low. We only allow alphanumeric because we are using “/” as a delimiter for our commands and in that case, it is a feature flaw where we should consider other delimiters. In this case, it does not bring major inconvenience to users as alternatives can be used eg. simply using their given name and so we believe it should be of low severity as it "is unlikely to affect normal operations of the product" as per the definition of severity.Low

Items for the Tester to Verify

:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]