search

nus-cs2103-AY2021S2 / pe-dev-response

0 stars 0 forks source link

The application accepts same phone number #3026

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Command Sequences: add -n John Doe -p 98765432 -e johnd@example.com -c Google -j Software Engineer -a 311, Clementi Ave 2, #02-25 -r University friend -t friends -t owesMoney add -n ALKK -p 98765432 -e johnd@example.com -c Google -j Software Engineer -a 311, Clementi Ave 2, #02-25 -r University friend -t friends -t owesMoney

Explaination: Commonly, the contact book should not accept two people with the same phone number.

Screenshot:

image.png

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

darkdestry-t commented 3 years ago

Team's Response

This issue is not considered an issue by the team. Our application is targeted towards business contact management.

This means that multiple employees can share the same office numbers if the office only has one landline, and the call would then be forwarded internally.

Similarly, the address and email address of different persons from the same company can be the same company address and email address (especially when the company is small and share the same email address for public communication).

This "uniqueness" issue is a complex issue and enforcing it would mean restricting use cases like the one mentioned above. This is not in the scope of this module and thus marked as NotInScope.

Duplicate status (if any):

Duplicate of #127