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

Wrong valid inputs for name in UG #1

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

Wrong valid inputs for name in UG #1

ncduy0303 opened this issue Nov 17, 2023 · 1 comment

Comments

@ncduy0303
Copy link
Owner

ncduy0303 commented Nov 17, 2023

Description

Trying to add a new person

Steps to reproduce

  1. Launch the application for the first time to load initial data.
  2. Run add n/La Niña 2 m/Computer Science y/2 e/johnd@u.nus.edu d/I’m a Frontend Developer t/06 t/19 sm/https://www.linkedin.com/in/john-doe-123456789 sm/https://www.linkedin.com/in/john-doe-123456789 nt/local g/m.

Expected behaviour

The person should be added successfully as this is sample valid input from the User Guide (same issue for name Élise DuPont)

Actual behaviour

An error is raised Names should only contain alphabetical characters and spaces, and it should not be blank. It should also not exceed 30 characters.

Reason for severity

Likely documentation issue (wrong valid inputs) since it is stated "Character Restrictions: Only alphabetical characters (A-Z, a-z) and spaces are allowed"

@soc-pe-bot
Copy link

soc-pe-bot commented Nov 20, 2023

Team's Response

We set the severity as very low as we believe it does not affect the usage of the app. For people who have symbols in their name, they can type their name without it, so the app would still work for them.

Items for the Tester to Verify

❓ Issue severity

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

  • I disagree

Reason for disagreement: Invalid examples.png

It seems like this bug is not a minor typo (cosmetic issue) to be classified as VeryLow severity, more like incorrect information given in the examples.

This cause confusion for users when they try out the app with the example names (La Niña and Élise DuPont) given in the User Guide, and thus minor inconvenience during app usage in rare occasions. Hence, Low severity.


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