nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Duplication Handling for wedding #3216

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

To reproduce bug, simply enter "addw n/Jack and Jill d/12/12/2024 c/1 2 3" with spaces in between the names.

While it mentioned that the names are case sensitive, it is not mentioned if white spaces will affect the duplicity checks. While this is possible for contacts since you have an additional phone number to do the checking, this allows a tired wedding planner to accidentally add multiples of the same wedding.

Screenshot 2024-11-15 at 4.48.52 PM.png


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

maertan commented 1 week ago

Team's Response

This issue is not in scope because fixing this is not essential for the app to be reasonably useful. Thus, according to the guidelines, it can be claimed as NotInScope. A user who accidentally added multiple spaces can easily rectify this using editw.

Severity should be low because this issue does not affect normal operations of the app, and only appears in rare cases where user made this mistake.

image.png

Duplicate status (if any):

--