nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Editing person allows changing names to a number (e.g phone number) #5367

Open nus-se-script opened 5 months ago

nus-se-script commented 5 months ago

Outline In your UG you outlined that `For , capitalization (e.g. ) or extra/leading/trailing spaces does not affect the value (e.g. ). The NAME should not have special characters (e.g.

, @, ! etc.).`

Names should not have numbers (otherwise people can just enter a phone number accidentally and TutorPro will allow that in the name field).

Steps to reproduce

  1. Ensure that there is at least 1 person in the list
  2. Enter Edit 1 n/1
  3. TutorPro will allow this name change

Why low severity If users are careful, this is not an issue. However carelessness may mean users enter phone number in the name field which TutorPro will allow. Users can still continue to use the app regardless, thus low severity.

Screenshot Screenshot 2024-04-19 at 4.35.54 PM.png


[original: nus-cs2103-AY2324S2/pe-interim#5215] [original labels: type.FunctionalityBug severity.Low]

tamagochuuu commented 5 months ago

Team's Response

Fixing this issue is of a lower priority than the work already done in v1.4.

Duplicate status (if any):

--