LWS49 / pe

0 stars 0 forks source link

Using empty phone prefix with edit results in error #8

Open LWS49 opened 5 months ago

LWS49 commented 5 months ago

image.png According to UG, this would be possible for any field, since it is not specified that this can only be done for any specific field.

image.png However, when done with p/, an error message for phone pops up.

soc-se-bot commented 5 months ago

Team's Response

image.png

Based on the parameter constraints given in the UG, the minimum length of the phone number is 1.

We agree that we could have made it clearer in the UG that the phone number is a mandatory field and cannot be removed.

However, we think that it should be reclassified as a Documentation bug instead, as the feature is indeed working as intended. (i.e. the user is unable to remove mandatory fields). We just neglected to mention that phone number is one of those mandatory fields.

Additionally, we think that this is a very rare situation as most users would be interested in keeping contact information in an address book application.

Thus, we have accepted the bug, but downgraded the severity from Medium --> Low, and changed the type from FuncionalityBug --> DocumentationBug.

Items for the Tester to Verify

:question: Issue type

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

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]