search

KamJiaYue / pe

0 stars 0 forks source link

The "Name" parameter is not general enough #3

Open KamJiaYue opened 12 months ago

KamJiaYue commented 12 months ago

Steps to reproduce:

  1. Enter ''add n/Aaron Tan Jun-Jie id/S8943782H p/98114839 e/example@mail.com 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 commented 12 months ago

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