search

saajidshaik02 / pe

0 stars 0 forks source link

[DG] Future Enhancements - Why no undo/redo + more features #6

Open saajidshaik02 opened 1 week ago

saajidshaik02 commented 1 week ago

For future enhancements, perhaps consider looking to add more features to your app ( this is part of future enhancements too!). Currently, it only states proposals to fix current issues in your app or extend features to be more complex.

For example, I feel the lack of undo or redo command is very troublesome. Let's take a look at the below video


video:https://raw.githubusercontent.com/saajidshaik02/pe/main/files/e1687b41-a051-4c88-ab5c-2029e673b4ef.mp4

Suppose I edit index 1 name to Saajid and want to revert it back. I need to scroll to find my name, which is sorted in alphabetical order (harder than normal to find, but still ok). You mentioned that your target audience can type very fast but clicking to find the contact and then clicking back to inputbox is definitely an inconvenience for them. (still a small issue + mentioned in known issues)

Now, I need to retype the exact index and change it back to the previous name which as I'm writing this now, cannot remember at all what it used to be. (again another small issue because maybe you have good memory)

But all these little inconveniences will give rise to harder usability of the app which in my opinion, results in a medium severity for not having such a feature for your target audience.

Not having a feature isn't considered a bug, but the fact it substantially affects your app's ability to meet requirements of your target audience is why undo/redo was the proposed feature under AB3 -> especially since your app is contact detail app replica.

(Its in proposed feature only because some other groups can change their app to a point where the undo and redo feature didn't have much efficiency benefit or they modified the undo and redo to a less general feature and for more user specific tasks only)

Beside undo/redo, perhaps other features like listing by XXXX order or finding based on other user details could potentially be a nice future nehancement to suggest so users can hope to expect more in the future and continue using your app as their wait for next patches.

image.png

nus-se-bot commented 5 days ago

Team's Response

We thank you for your detailed response and recommendations. It is possible for additional features to come in the future. In response to the lack of an undo/redo feature, we recognise that it may cause inconvenience to some users.

However, most changes to the address book are easily rectifiable. For example in the issue you mentioned, the message bar displays the edited contact, which includes the new contact name: Saajid. Hence the user can simply use edit Saajin n/Alexandra Haas, or execute find n/Saajin to narrow down the search scope and change the name back using edit.

Hence this does not really affect usage by users (as described by our user stories), and we are changing the severity to low. In addition, for an address app for managing contacts of CCA members, additional features such as redo/undo are not as important as the other features that we have implemented (pagination, find, roles, nicknames, etc), hence this is not in scope.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]