ezeAng / pe

0 stars 0 forks source link

Unable to delete or edit a contact name or details - also leads to trickle down effects for users using the app for a longer time. #6

Open ezeAng opened 1 year ago

ezeAng commented 1 year ago

There is an issue as there is no way for a user to make changes to his contact list if he made an error without having to clear the entire event and contact schedule.

In the situation where a user added a contact number wrongly, say a mismatched name to contact number.

Actual: James - 91234567

Mabel - 81231231

What user input: Mabel - 91234567

James - 81231231

Then there is no way to change the name or contact number by editing. If the user has a large contact base built over time, it would be detrimental to have to clear the entire database including events just to change a name or phone number, a small detail. Else he/she would have to live with the error moving forward and possibly cause more confusion.

nus-pe-bot commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Missing feature to delete a contact

Description

Unable to delete individual contacts, which impedes maintaining a proper contact list.

It would be common for a user to want to keep their contact list organised, and delete contacts for events that are finished, or contacts that they had a bad experience with. The only obvious work-around for this is to run the clear command, which is insufficient deleting only 1 specific contact is desired.


[original: nus-cs2103-AY2223S2/pe-interim#3218] [original labels: severity.High type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

This is already specified in the DG under Appendix: Planned Enhancements as our future enhancements.

image.png

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue response Team chose [`response.NotInScope`] - [x] I disagree **Reason for disagreement:** This issue has been reported by other testers as well, but my reasons for thinking this is a severe bug may be different. For this app, which does not allow contact deletion or editing, there is no way to reverse what you do without clearing all events and contacts. The purpose of the app is to facilitate and improve contact management and not cause further inconvenience, however, when mistakes are made, a user will have to **delete all the contacts** and whats more **delete all the events**. What this means, for a user is that if he decided to use your app for a longer time and had amassed many events and many contacts, if he made a mistake, it would not be reversible and to make 1 small change, he would have to delete the entire database with the clear command and subsequently **repopulate the entire database** with -- and what if he makes a mistake while repopulating? You guessed it, delete all events and all contacts, and repopulate. This also means you expect the user to perform no mistakes over his entire lifetime of using the app to add contacts or be comfortable with constantly having an error-full contact database, which I think is extremely unlikely. For that reason I think that this bug, causes much more inconvenience than benefits and defeats the purpose of using the application and hence should be a reasonable bug. I would also argue that due to this, the application released was not even a complete one, since it cannot perform basic commands despite being able to do higher-level commands. Lastly, the group has more than once put issues into "planned enhancements", I do not think it is fair to claim everything as a "planned enhancement" for example, this issue here is a clear bug.