lzf834 / pe

0 stars 0 forks source link

No character limit for phone numbers #3

Open lzf834 opened 2 years ago

lzf834 commented 2 years ago

image.png

image.png

It is an unrealistic scenario and does not appear to break the functionality of the application. Therefore, severity is set to verylow.

nus-pe-script commented 2 years ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Long Phone numbers on buyerrs not shown fully

image.png

To replicate:

  1. add a buyer with a very long phone number

The phone number shown in the card ends with ellipses and clicking on the card does not reveal the full information/Phone number of the buyer. This may prevent the user from obtaining the phone number of the buyer should he/she wants to contact him/her.


[original: nus-cs2103-AY2122S2/pe-interim#1878] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

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

image.png

Please refer to the above. For more information, please refer to https://nus-cs2103-ay2122s2.github.io/website/admin/tp-pe.html#pe-phase-1-bug-reporting-2

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: The other issue is one where the information card does not fully show the phone number of the user in its entirety. This issue revolves around allowing long inputs into the phone number attribute.


:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: image.png

Allowing any input more than 15 characters from phone number seems to be unrealistic.


:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]