Skip to content

#27 incorrect summary count for daily / monthly income, expense trans…#370

Merged
gjong merged 1 commit intomasterfrom
hotfix/27-count-incorrect
Mar 13, 2026
Merged

#27 incorrect summary count for daily / monthly income, expense trans…#370
gjong merged 1 commit intomasterfrom
hotfix/27-count-incorrect

Conversation

@gjong
Copy link
Copy Markdown
Contributor

@gjong gjong commented Mar 13, 2026

Breaking change

Proposed change

Fix for bug in summary in transaction list header.

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (which adds functionality to the application)
  • Deprecation (breaking change to happen in the future)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Additional information

Checklist

  • The code change is tested and works locally.
  • Local tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 9.73% 314 / 3226
🔵 Statements 9.56% 329 / 3439
🔵 Functions 6.21% 100 / 1608
🔵 Branches 8.72% 127 / 1455
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/components/account/transaction-list.component.tsx 0% 0% 0% 0% 20-89
src/components/transaction/transaction-detail.component.tsx 0% 0% 0% 0% 32-164
src/core/index.jsx 13.04% 0% 0% 13.04% 10-14, 29-70
Generated in workflow #1485 for commit 113382d by the Vitest Coverage Report Action

@gjong gjong merged commit 21576e6 into master Mar 13, 2026
7 checks passed
@gjong gjong deleted the hotfix/27-count-incorrect branch March 13, 2026 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Daily total is always 0 in transaction overview

1 participant