vishnuvk47 / pe

0 stars 0 forks source link

Excess white space between name is not trimmed #3

Open vishnuvk47 opened 1 year ago

vishnuvk47 commented 1 year ago

Screenshot 2023-04-14 at 4.25.34 PM.png

Refer above, as highlighted

nus-pe-bot commented 1 year ago

Team's Response

White spaces in the name is intended. Allows for user's full name.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: In the highlighted name "biomedical..engineering", there are two subsequent spaces, which are not typically found in full names. While it was mentioned in the rebuttal that this design allowed users to enter their full name, it could be argued that the presence of multiple spaces in a name is unnecessary and could lead to confusion. This issue could be easily addressed by implementing measures such as splitting the name by spaces or asking for the first name and last name separately, trimming white spaces, and concatenating them. Therefore, it may not be appropriate to completely reject this issue, and it could be assigned a low severity rating, as it is a relatively minor issue that can be resolved easily.

I also made an error in identifying this as a functionality bug when this should have been a feature flaw.