tankh99 / pe

0 stars 0 forks source link

UG states that there is no limitation on phone number digits, but Application does otherwise #2

Open tankh99 opened 2 months ago

tankh99 commented 2 months ago

It is false that there is no limitation on the number of digits, as there is a validation error message in the application that says otherwise

Screenshot 2024-04-19 at 4.17.22 PM.png

Screenshot 2024-04-19 at 4.17.59 PM.png

soc-pe-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Mismatch between UG and Error Message for PHONE_NUMBER

In the UG, it is stated that "there are no limitations on the number of digits" for the PHONE_NUMBER parameter.

image.png

However, when adding a student with a two digit phone number, an error is thrown. Command used:

add n/Jasdasdhn Doe p/22 e/johnd@example.com y/2 m/Computer Science tg/johndoe r/Very quiet student g/TUT04 g/LAB05 

image.png


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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

No details provided by team.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue type Team chose [`type.FunctionalityBug`] Originally [`type.DocumentationBug`] - [x] I disagree **Reason for disagreement:** I don't really agree that this is a functionality bug because this is more so a documentation issue. There was nothing wrong functionality-wise, just information was not complete