search

petermonky / pe

0 stars 0 forks source link

Input value for name does not accept "/" #2

Open petermonky opened 2 years ago

petermonky commented 2 years ago

image.png

The legal name of a person may contain the "/" character; by prohibiting this character, the functionality of the application is limited.

Expected: Successful execution

Actual: Failed execution

nus-se-bot commented 2 years ago

Team's Response

While it is a fair point that some names may contain special symbols, friendly names like nicknames are satisfactory for our application. Especially because the clients' names are simply personal records for the financial advisor, the "/" is not required for the application to be functional

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: It is up to the user to decide how to name their clients in the application; althrough friendly names are a possiblity, to users who wish to save their clients with their legal names, the "/" may be a necessity. Understandably, the "/" is expected to be rare in clients' names, so perhaps a severity of severity.Low and bug type of FeatureFlaw is more appropriate.

https://nus-cs2103-ay2122s2.github.io/website/schedule/week13/project.html#functionality-bugs-2

image.png