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

The "Name" parameter is not general enough #3

Open
KamJiaYue opened this issue Nov 17, 2023 · 1 comment
Open

The "Name" parameter is not general enough #3

KamJiaYue opened this issue Nov 17, 2023 · 1 comment

Comments

@KamJiaYue
Copy link
Owner

Steps to reproduce:

  1. Enter ''add n/Aaron Tan Jun-Jie id/S8943782H p/98114839 e/[email protected] a/Serangoon HDB 123''
  2. Hit ''Enter''

Expected: The system should output this error message: "Names should only contain alphanumeric characters and spaces, and it should not be blank".

Actual: Same as expected.

Potentially Problematic Behaviour:
Although the user guide mentioned about names should only contain alphanumeric characters, there are people whose names contains a dash ("-"). For patients whose names contains dashes, users might encounter an issue while adding patients data into the system.

Suggestions:
Perhaps, removing the constraint such that names could also contain non alphanumeric characters would solve this issue.

@soc-pe-bot
Copy link

Your response not required for this bug as the team has accepted the bug as it is.

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