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.

Sign up for GitHub

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

Jump to bottom

Dashboard Transaction Statistics erroneously includes deleted transactions #6102

Closed
1 task done
tactilis opened this issue Jul 13, 2023 · 1 comment
Closed
1 task done

Dashboard Transaction Statistics erroneously includes deleted transactions #6102

tactilis opened this issue Jul 13, 2023 · 1 comment
Labels
bug fixed
Milestone
v1.7.0

Comments

@tactilis
Copy link

MMEX version:

  • 1.6.4

The Dashboard includes a panel named Transaction Statistics which displays the number of transactions marked as Follow Up and the Total number of transactions.

image

Actual Result

Both of these numbers include a count of transactions that have been deleted and are in the Deleted Transactions view.

Expected Result

Deleted transactions should be excluded from the counts.
@vomikan vomikan added the bug label Jul 13, 2023
@vomikan vomikan added this to the v1.7.0 milestone Jul 13, 2023
n-stein added a commit to n-stein/moneymanagerex that referenced this issue Jul 13, 2023
@n-stein
fix(moneymanagerex#6102): exclude deleted txns from homepage counts
bdc9d5b
@n-stein
Copy link
Contributor

n-stein commented Jul 13, 2023

fixed

vomikan added a commit that referenced this issue Jul 13, 2023
@vomikan
Merge pull request #6106 from n-stein/fix(#6102)
b5afbc3 
fix(#6102): exclude deleted txns from homepage counts
@vomikan vomikan added the fixed label Jul 13, 2023
@vomikan vomikan closed this as completed Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug fixed
Projects
None yet
Milestone
v1.7.0
Development

No branches or pull requests
3 participants
@tactilis @vomikan @n-stein