[PR #1988] [MERGED] Custom Reports optimization #4057

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/1988
Author: @carkom
Created: 11/28/2023
Status: Merged
Merged: 12/6/2023
Merged by: @carkom

Base: masterHead: reportsOptimize


📝 Commits (10+)

📊 Changes

23 files changed (+882 additions, -716 deletions)

View changed files

📝 packages/desktop-client/src/components/reports/ChooseGraph.tsx (+30 -52)
📝 packages/desktop-client/src/components/reports/ReportOptions.tsx (+132 -0)
📝 packages/desktop-client/src/components/reports/ReportTable.tsx (+22 -9)
📝 packages/desktop-client/src/components/reports/ReportTableHeader.tsx (+88 -66)
📝 packages/desktop-client/src/components/reports/ReportTableList.tsx (+4 -4)
📝 packages/desktop-client/src/components/reports/ReportTableTotals.tsx (+100 -86)
📝 packages/desktop-client/src/components/reports/graphs/AreaGraph.tsx (+4 -4)
📝 packages/desktop-client/src/components/reports/graphs/BarGraph.tsx (+3 -7)
📝 packages/desktop-client/src/components/reports/graphs/BarLineGraph.tsx (+2 -10)
📝 packages/desktop-client/src/components/reports/graphs/DonutGraph.tsx (+4 -8)
📝 packages/desktop-client/src/components/reports/graphs/LineGraph.tsx (+2 -5)
📝 packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx (+4 -5)
📝 packages/desktop-client/src/components/reports/reports/CustomReport.js (+36 -20)
📝 packages/desktop-client/src/components/reports/reports/CustomReportCard.js (+7 -1)
📝 packages/desktop-client/src/components/reports/spreadsheets/default-spreadsheet.tsx (+110 -427)
packages/desktop-client/src/components/reports/spreadsheets/filterHiddenItems.ts (+24 -0)
packages/desktop-client/src/components/reports/spreadsheets/grouped-spreadsheet.ts (+141 -0)
packages/desktop-client/src/components/reports/spreadsheets/makeQuery.ts (+86 -0)
packages/desktop-client/src/components/reports/spreadsheets/recalculate.ts (+69 -0)
📝 packages/desktop-client/src/components/reports/util.ts (+0 -11)

...and 3 more files

📄 Description

Custom Reports can be very slow for larger budget files. I've remade the data pulls to speed things up.


🔄 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/1988 **Author:** [@carkom](https://github.com/carkom) **Created:** 11/28/2023 **Status:** ✅ Merged **Merged:** 12/6/2023 **Merged by:** [@carkom](https://github.com/carkom) **Base:** `master` ← **Head:** `reportsOptimize` --- ### 📝 Commits (10+) - [`465fa51`](https://github.com/actualbudget/actual/commit/465fa51ebdfd9486ecc1e8f2ef755891d118655a) Range fix and payee fix - [`1074e06`](https://github.com/actualbudget/actual/commit/1074e06d24d6770197d3e380423dc62dab0df82f) bug fixes and UI tweaks - [`2a3b3ab`](https://github.com/actualbudget/actual/commit/2a3b3ab9b44608c178a32f5b099d4a4ab3a8c5c6) range options, hover UI - [`92cbddd`](https://github.com/actualbudget/actual/commit/92cbddd9ac9bbb42d0aebebca09ec84e23626417) Select - UnSelect All Buttons - [`aa0a12e`](https://github.com/actualbudget/actual/commit/aa0a12ea1f0d4cfc80ba93a1871310a70251e0c2) fix hidden group bug - [`0784d2d`](https://github.com/actualbudget/actual/commit/0784d2d75f05cb7eea0c1289ed96404499c42011) YAxis PrivacyFilter - [`68c3f96`](https://github.com/actualbudget/actual/commit/68c3f96f499ef9a051be3a294f1b51cfeb3c86e4) notes - [`71ca78d`](https://github.com/actualbudget/actual/commit/71ca78df9b2c9221388cac39685412fb7f426e72) more privacyFilter graphs - [`3a679f8`](https://github.com/actualbudget/actual/commit/3a679f808e9dac42d6a30a5334ca44f0e0eda0c5) Merge branch 'master' into reportBugs - [`ae8f247`](https://github.com/actualbudget/actual/commit/ae8f2472f0612eff1089244c7f054ca7997a2469) overflowY fix ### 📊 Changes **23 files changed** (+882 additions, -716 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/reports/ChooseGraph.tsx` (+30 -52) 📝 `packages/desktop-client/src/components/reports/ReportOptions.tsx` (+132 -0) 📝 `packages/desktop-client/src/components/reports/ReportTable.tsx` (+22 -9) 📝 `packages/desktop-client/src/components/reports/ReportTableHeader.tsx` (+88 -66) 📝 `packages/desktop-client/src/components/reports/ReportTableList.tsx` (+4 -4) 📝 `packages/desktop-client/src/components/reports/ReportTableTotals.tsx` (+100 -86) 📝 `packages/desktop-client/src/components/reports/graphs/AreaGraph.tsx` (+4 -4) 📝 `packages/desktop-client/src/components/reports/graphs/BarGraph.tsx` (+3 -7) 📝 `packages/desktop-client/src/components/reports/graphs/BarLineGraph.tsx` (+2 -10) 📝 `packages/desktop-client/src/components/reports/graphs/DonutGraph.tsx` (+4 -8) 📝 `packages/desktop-client/src/components/reports/graphs/LineGraph.tsx` (+2 -5) 📝 `packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx` (+4 -5) 📝 `packages/desktop-client/src/components/reports/reports/CustomReport.js` (+36 -20) 📝 `packages/desktop-client/src/components/reports/reports/CustomReportCard.js` (+7 -1) 📝 `packages/desktop-client/src/components/reports/spreadsheets/default-spreadsheet.tsx` (+110 -427) ➕ `packages/desktop-client/src/components/reports/spreadsheets/filterHiddenItems.ts` (+24 -0) ➕ `packages/desktop-client/src/components/reports/spreadsheets/grouped-spreadsheet.ts` (+141 -0) ➕ `packages/desktop-client/src/components/reports/spreadsheets/makeQuery.ts` (+86 -0) ➕ `packages/desktop-client/src/components/reports/spreadsheets/recalculate.ts` (+69 -0) 📝 `packages/desktop-client/src/components/reports/util.ts` (+0 -11) _...and 3 more files_ </details> ### 📄 Description Custom Reports can be very slow for larger budget files. I've remade the data pulls to speed things up. --- <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:50:53 -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#4057