search

AlfredBeNoel / pe

0 stars 0 forks source link

Invalid checking for Team Name #9

Open AlfredBeNoel opened 11 months ago

AlfredBeNoel commented 11 months ago

Screenshot 2023-11-17 at 5.10.15 PM.png When doing newteam tn/0 tl/Bernice Yu

it doesn't give a proper error

nus-pe-bot commented 11 months ago

Team's Response

Thank you for pointing this out. However, it is indeed an expected behavior, as the user is allowed to name a team as "0". This gives more freedom for the user to name their team.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I had set this as a documentation bug as although this is expected behaviour to give more freedom to the user, it doesn't mention any of such limitations or checks to the user in the User Guide, and hence could cause confusion since there is no information about checking of the team's name that it checks for alphanumeric characters, but doesn't allow for blanks. This should have been mentioned in the UG.