nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Case sensitive names #1882

Open nus-pe-bot opened 1 day ago

nus-pe-bot commented 1 day ago

When adding names into NomNomNotifier, I was allowed to add two different people with similar names using the inputs,

add n/John Doe p/98765432 e/johnd@example.com a/John street, block 123, #01-01 pc/666234

followed by

add n/John doe p/98765432 e/johnd@example.com a/John street, block 123, #01-01 pc/666234

This resulted in there being 2 different persons in NomNomNotifier, John Doe, and John doe, which doesn't follow the real life logic where peoples name in real life aren't case sensitive

image.png

According to the CS2103T website, under feature flaws, this is considered as one, hence the categorisation of it

image.png


[original: nus-cs2103-AY2425S1/pe-interim#2587] [original labels: severity.Medium type.FeatureFlaw]

noth-k commented 10 hours ago

Team's Response

Thank you for bringing this up. We accept this as a valid issue, as the app currently allows adding two individuals with similar names that differ only in letter casing (e.g., "John Doe" and "John doe"). However, we have classified this as a low-severity issue, as it does not hinder any core functionality or prevent users from performing operations as intended.

In real-world scenarios, having two individuals with such similar names is exceedingly rare. Additionally, restaurants, our primary users use consistent casing when entering names into the system, as operational efficiency relies on clear and accurate data entry. This greatly reduces the likelihood of this issue arising during normal usage.

At most, this could cause minor inconvenience, and it can be easily resolved by users ensuring consistent casing during data entry. While we acknowledge this as a feature flaw, its impact on the app’s overall functionality and usability is minimal, and it does not interfere with the core tasks the app is designed to handle.

We appreciate your feedback and will consider this for future iterations to further enhance the user experience. Thank you for helping us improve!

Duplicate status (if any):

--