juliusgambe / pe

0 stars 0 forks source link

[FeatureFlaw] Deletion by name #5

Open juliusgambe opened 9 months ago

juliusgambe commented 9 months ago

Feature Flaw:

Deleting by name might be susceptible to accidental deletion of contacts with the same name but are differentiated only by their casing. ie. John doe and John Doe

As a result, you may want to delete John Doe but you accidentally wrote John doe instead resulting in you deleting the data for John doe instead of John Doe. Severity would be medium as this is likely to happen for duplicate names and the result would be catastrophic as the John doe's data is completely lost without even confirmation.

You may consider the original AB3 way of deleting by index.

Actual Output

Screenshot 2023-11-17 at 4.40.58 PM.png

Steps To Reproduce

  1. Open app
  2. Add a contact with name John doe and John Doe
  3. "Accidentally" delete delete n/John doe when you wanted to delete John Doe
nus-pe-script commented 9 months ago

Team's Response

Thank you for the feedback. However, we chose to allow users to delete users by name instead of index as the names are unique and it is much more convenient if you do have an address book of a substantial size (i.e. have to scroll down 200 contacts to find index of someone is very inconvenient).

Referring to your example, if you are unsure which john doe you are looking for, consider using the find command first, like you would for the deleting using index. We also think that a typo index is equally, if not more, likely to happen than a case typo in the contact name.

Additionally, regarding data loss resulting from a typo, it is stated in our planned enhancements that we plan to add an undo/redo feature to help with this.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]