twothicc / pe

0 stars 0 forks source link

Unable to add more than 1 phone number for a vendor #5

Open twothicc opened 2 years ago

twothicc commented 2 years ago

Vendors tend to have main contact numbers and other alternative contact numbers if say some member of the vendor's business is not contactable.

It makes sense for a front desk staff to be able to store multiple contact numbers for a single vendor in case of emergencies. However, your app does not provide such a feature and in fact, even blocks such inputs.

image.png

I would say that this is a feature flaw and impacts front desk staff who wish to use the app to manage a proper hotel management business.

soc-se-bot commented 2 years ago

Team's Response

This is not within the scope of our app. Reducing severity to Low.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: image.png

It says here in your user stories that your users should be able to input vendor details, which would include phone numbers as well.

image.png

The above also shows that you have a use case that aims to fulfill the above given user story. This means that add details of vendors is an expected feature for your v1.4 application.

My issue is that your application does not allow for users to input 2 or more contact numbers for a single vendor. This is a realistic concern as many vendors can have secondary contacts in case the primary contact is uncontactable. The hotel front desk staff may want to input 2 or more contact numbers for a single vendor to reduce chances of a vendor being uncontactable.

image.png

Given that your application blocks inputs of more than 1 contact number, I would say it is a case of overzealous input validation that causes your application to fall short of fulfilling the user story add details of vendors for your intended users.

Allowing more than 1 contact number would simply require you to change the validation check for vendor phone number, and should not require much additional effort

image.png

Given that my issue is related to features promised in v1.4 and that inputting vendor details can be implemented in a better way to fulfill the needs of the application's intended target users without much additional efforts, I feel that the issue should be accepted.


:question: Issue severity

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

Reason for disagreement: Given that it is very common for vendors to have secondary or even tertiary contact numbers, I believe that blocking inputs of more than 1 contact numbers will cause occasional inconvenience to the hotel front desk staff who will be using your application. That being said, the hotel front desk staff can still continue to use the application but have to carry the unnecessary risk of not being able to contact a vendor.

As such, I believe that the severity of such an issue should be Medium

image.png