search

hcs1203 / pe

0 stars 0 forks source link

Error message seems inaccurate #3

Open hcs1203 opened 1 year ago

hcs1203 commented 1 year ago

If it is a requirement that names should only contain alphanumeric characters and spaces (might just be alphabetic characters instead of alphanumeric character), it is better to make it one, else let the '/' be treated as normal, especially since a name that is just numbers are accepted with no errors

Screenshot 2023-11-17 at 4.22.32 PM.png Screenshot 2023-11-17 at 4.26.02 PM.png

nus-pe-script commented 1 year ago

Team's Response

Hi, I am the dev responsible for the fields validation rework, and hence I am responsible for this issue. Thanks for the report, however we've already covered this under our second planned enhancement in the DG. Hence I've chosen to reject this issue.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]