tobihy / pe

0 stars 0 forks source link

Contact list view is reset after `edit` command #6

Open tobihy opened 2 years ago

tobihy commented 2 years ago

After using find command to look for Roy, I used the edit command to change Roy's phone number to a different number.

Screenshot 2022-04-16 at 2.37.25 PM.png

However, after executing the command, the contact list view in the application would be reset to view all contacts, instead of remaining at the view with only Roy present.

Screenshot 2022-04-16 at 2.37.30 PM.png

This can cause an inconvenience to users of the application, especially so when the user has many contacts within the app (>100), and editing contacts would force him/her back to the full view. This is especially troublesome, if the user wishes to chain together many edit commands with Roy as the target contact, one at a time, but has to constantly look for Roy using the find command due to this issue.

This behaviour is also not specified in the User Guide.

soc-se-bot commented 2 years ago

Team's Response

Changed to very low because this is purely cosmetic.

It is very unlikely that user will have to edit multiple Roys, this to us is extreme user behaviour and should not be considered as a bug.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Given that such behaviour of the application is not specified in the user guide, it is fair for such an occurrence to be considered as a bug, rather than a "extreme user behaviour" that is "purely cosmetic".

The user would run into this inconvenience when editing contacts, which would affect their productivity when using the application. This is especially painful when the contacts list is long, as the user would have to look for the same contact amidst a large number of contacts.


:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Medium]

Reason for disagreement: As the edit function is one of the four main CRUD functions, it is expected that the user would use the edit command frequently.

Hence, the user would run into this inconvenience a lot of the time, which would affect their productivity when using the application.

As such, given the importance of the edit command as the update function for the application, the bug deserves severity.Medium.