imanamirshah / pe

0 stars 0 forks source link

`adduser` auto-capitalise the input #4

Open imanamirshah opened 7 months ago

imanamirshah commented 7 months ago

image.png

I input my name as tester however, the program registers and saves it as Tester.

nus-se-script commented 7 months ago

Team's Response

Works as intended: All user names are automatically set to capitalize the first letter and have the others be lowercase. This makes it easier for the user to enter the user name without having to worry about capitalization.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: It is weird that you say this is intended. Your user guide clearly shows the expected output to take the name exactly as it is when using this command. I've attached the screenshot of your user guide for you below. If this was indeed intended, don't you think it should be reflected here in the expected output, since that's what users should expect to happen?

image.png

Also, some names do not start with a capital letter. For example, Leonardo da Vinci's middle name starts with a small letter. So does Vincent van Gogh's. What if they want to use their middle name to add a user?