search

officialchengyud / pe

0 stars 0 forks source link

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

Open officialchengyud opened 2 months ago

officialchengyud commented 2 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

nus-se-bot commented 2 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

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

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your reason]