theeugenechong / pe

0 stars 0 forks source link

Allow slashes in students names #5

Open theeugenechong opened 2 years ago

theeugenechong commented 2 years ago

Names like shown below should be accepted as they are real names.

image.png

nus-pe-bot commented 2 years ago

Team's Response

Feature request. Does not affect usage.

Accepting slashes may cause issues to the parser as parameters can be in any order (for flexibility). Problem can be easily fixed by changing s/o to so or s-o.

For example: (if slashes are allowed)

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]