[PR #2777] [MERGED] Filter Balance fix #4507

Closed
opened 2026-02-28 20:56:48 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/2777
Author: @carkom
Created: 5/18/2024
Status: Merged
Merged: 5/29/2024
Merged by: @carkom

Base: masterHead: filterQuery


📝 Commits (10+)

📊 Changes

7 files changed (+37 additions, -9 deletions)

View changed files

📝 packages/desktop-client/e2e/rules.test.js-snapshots/Rules-creates-a-split-transaction-rule-and-makes-sure-it-is-applied-when-creating-a-transaction-1-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/rules.test.js-snapshots/Rules-creates-a-split-transaction-rule-and-makes-sure-it-is-applied-when-creating-a-transaction-2-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/rules.test.js-snapshots/Rules-creates-a-split-transaction-rule-and-makes-sure-it-is-applied-when-creating-a-transaction-3-chromium-linux.png (+0 -0)
📝 packages/desktop-client/src/components/accounts/Account.jsx (+25 -0)
📝 packages/desktop-client/src/components/accounts/Balance.jsx (+4 -8)
📝 packages/desktop-client/src/components/accounts/Header.jsx (+2 -1)
upcoming-release-notes/2777.md (+6 -0)

📄 Description

On the accounts page - filter balance only adds up transactions that are showing. If your filter has more than one page it won't be added to the balance unless you scroll to the bottom and reveal all transactions. This fixes that.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/actualbudget/actual/pull/2777 **Author:** [@carkom](https://github.com/carkom) **Created:** 5/18/2024 **Status:** ✅ Merged **Merged:** 5/29/2024 **Merged by:** [@carkom](https://github.com/carkom) **Base:** `master` ← **Head:** `filterQuery` --- ### 📝 Commits (10+) - [`bbca4da`](https://github.com/actualbudget/actual/commit/bbca4da3a63743627765d33555fd9093ab2abed0) filterQuery - [`31d485b`](https://github.com/actualbudget/actual/commit/31d485b2895f1658fc63fdee248d5cdd55c43f03) notes - [`1f6a7b8`](https://github.com/actualbudget/actual/commit/1f6a7b865e02f0008be266ec5e1173dac5a9db31) Merge branch 'master' into filterQuery - [`d99adc5`](https://github.com/actualbudget/actual/commit/d99adc51c872840a9ed065908e7e936bc13930a1) Merge branch 'master' into filterQuery - [`7529f90`](https://github.com/actualbudget/actual/commit/7529f908c536144d7d7ce25ad8c16bd46d4a5a26) update - [`16c1466`](https://github.com/actualbudget/actual/commit/16c1466e565a3a68dd44fc4149056472fe9e7d10) lint fix - [`e3de84a`](https://github.com/actualbudget/actual/commit/e3de84ae39ea97c076dad86e9d73eaec3a129f5c) VRT - [`ecca602`](https://github.com/actualbudget/actual/commit/ecca602639f3f147fb9ee641f11f63b247d99d91) Merge branch 'master' into filterQuery - [`0427316`](https://github.com/actualbudget/actual/commit/04273169701b7cea894e5777be3bc21cbfc4cb99) fix account filter - [`f70962c`](https://github.com/actualbudget/actual/commit/f70962cf2529da7d10d45d7eaae8fb536adad4a7) VRT ### 📊 Changes **7 files changed** (+37 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/e2e/rules.test.js-snapshots/Rules-creates-a-split-transaction-rule-and-makes-sure-it-is-applied-when-creating-a-transaction-1-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/rules.test.js-snapshots/Rules-creates-a-split-transaction-rule-and-makes-sure-it-is-applied-when-creating-a-transaction-2-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/rules.test.js-snapshots/Rules-creates-a-split-transaction-rule-and-makes-sure-it-is-applied-when-creating-a-transaction-3-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/src/components/accounts/Account.jsx` (+25 -0) 📝 `packages/desktop-client/src/components/accounts/Balance.jsx` (+4 -8) 📝 `packages/desktop-client/src/components/accounts/Header.jsx` (+2 -1) ➕ `upcoming-release-notes/2777.md` (+6 -0) </details> ### 📄 Description On the accounts page - filter balance only adds up transactions that are showing. If your filter has more than one page it won't be added to the balance unless you scroll to the bottom and reveal all transactions. This fixes that. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-02-28 20:56:48 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#4507