Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Special symbol not allowed in client name #8

Open
yunbinmo opened this issue Apr 16, 2022 · 1 comment
Open

Special symbol not allowed in client name #8

yunbinmo opened this issue Apr 16, 2022 · 1 comment

Comments

@yunbinmo
Copy link
Owner

yunbinmo commented Apr 16, 2022

image.png
Client names containing s/o cannot be added, which can cause problems since this is an app for financial advisor, it would be important to record the client's legal name.

@nus-se-bot
Copy link

nus-se-bot commented Apr 19, 2022

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

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Input value for name does not accept "/"

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


[original: nus-cs2103-AY2122S2/pe-interim#2938] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

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

❓ Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

  • I disagree

Reason for disagreement: [replace this with your explanation]


❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: [replace this with your explanation]


❓ Issue severity

Team chose [severity.Medium]
Originally [severity.Low]

  • I disagree

Reason for disagreement: [replace this with your explanation]


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants