L1uY1jun / pe

0 stars 0 forks source link

Person name bug #5

Open L1uY1jun opened 1 year ago

L1uY1jun commented 1 year ago

*Bug Description:

In general, legal names can include / - @ , ' .such characters, however, names with such characters are rejected in this application.

Expected:

Adding a person named Marthin Luther jr. should be possible with the command:

add-person n/Marthin Luther jr. e/mathin@example.com p/99174917

Actual:

However, such a name is rejected, which can cause quite a bit of inconvenience unless you are intending to not include part of the name

image.png

nus-se-script commented 1 year ago

Team's Response

Firstly, we deem that this is a feature flaw rather than functionality as this bug is targeting our design considerations rather than the fact that our product is not working as specified. This is because we did explicitly mention in our UG that we only allow our names to be only alphanumeric characters.

Secondly, we deem that this bug should have only a low severity, especially when taking into account the nature of our application. This is because unlike other applications like a hotel guest tracking application (etc), our app does not have a heavy official use case to track people's names-the use of names in our app is merely a tool for users to identify their friends with. Thus, even in instances where you have a friend whose name has a special symbol, the user could always circumvent it by naming his friend in a way that does not require the use of special symbols. For example, consider the name “José Murinho”. Naming him as “Jose Murinho” also serves the purpose of being able to identify who the contact is well enough. Thus, since this is a flaw that is unlikely to affect normal operations of the product, we deem that it should have a severity of Low.

However, we would also like to extend our gratitude to you for pointing this out to us, this is a good point that we feel we can work on in future iterations.

Items for the Tester to Verify

: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.Medium]

Reason for disagreement: [replace this with your explanation]