search

triciiaaa / pe

0 stars 0 forks source link

Name field does not accommodate for special characters #4

Open triciiaaa opened 4 months ago

triciiaaa commented 4 months ago

Description

Names with special characters such as the one shown in the screenshot are not allowed although it is a valid name found on the Internet.

Steps to reproduce

  1. input add n/Pedro de Alcântara João Carlos Leopoldo Salvador Bibiano Francisco Xavier de Paula Leocádio Miguel Gabriel Rafael Gonzaga de Habsburgo-Lorena e Bragança p/98765432 e/johnd@example.com m/Computer Science

Expected

Person should be added successfully into the application.

Actual

Error message that name fields should only consist of alphanumeric characters and spaces is shown.

Screenshots

image.png

nus-pe-script commented 4 months ago

Team's Response

image.png

As shown in the DG, we put it under Planned Enhancements as shown here: https://ay2324s2-cs2103t-f13-4.github.io/tp/DeveloperGuide.html#enhanced-name-input

We noticed this mistake as well and we will implement this in future versions of the product!

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Thanks for your response! I appreciate that the team will implement this in future versions of the product. In that case, would it be more suitable for the response label to be response.NotInScope instead since it is possible to work on and already planned for future iterations? Do refer to the response labels and the corresponding descriptions below.

image.png