Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

History feature flaw for monetary commands #4179

Open
nus-pe-bot opened this issue Nov 15, 2019 · 1 comment
Open

History feature flaw for monetary commands #4179

nus-pe-bot opened this issue Nov 15, 2019 · 1 comment

Comments

@nus-pe-bot
Copy link

Start with 0 savings and 30 budget:

image.png

save 10:

image.png
save 10 (again):

image.png
history:

image.png

Note that there is only 1 save 10 command. Although they are the same command, as a user I need to see multiples of my save commands, even if i save the same amount each time.


[original: nus-cs2103-AY1920S1/pe-interim#4201]
@fatclarence
Copy link
Contributor

fatclarence commented Nov 18, 2019

Team's Response

Thank you for your feedback on our application!

We would like to accept this as a valid feature flaw because it makes sense for the user/developer to see multiple save commands, especially if he/she is unsure if he/she has already made a saving. In fact, as an extension to this issue, we should also reconsider the History Command to include functionalities that allow the user to review previous typed purchases because these would also be essential for the users to review.

With that, we will be accepting this bug, and we will work on it as it will affect the users/developers experience on an occasional basis. Thank you!

Duplicate status (if any):

--

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants