[PR #2726] [MERGED] Custom Reports: TS strict changes #1 #46296

Closed
opened 2026-04-26 08:23:08 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/2726
Author: @carkom
Created: 5/8/2024
Status: Merged
Merged: 5/13/2024
Merged by: @carkom

Base: masterHead: CR-strict1


📝 Commits (5)

📊 Changes

20 files changed (+253 additions, -167 deletions)

View changed files

📝 packages/desktop-client/src/components/reports/ChooseGraph.tsx (+4 -3)
📝 packages/desktop-client/src/components/reports/ReportSummary.tsx (+8 -9)
📝 packages/desktop-client/src/components/reports/graphs/AreaGraph.tsx (+3 -3)
📝 packages/desktop-client/src/components/reports/graphs/BarGraph.tsx (+7 -7)
📝 packages/desktop-client/src/components/reports/graphs/DonutGraph.tsx (+3 -3)
📝 packages/desktop-client/src/components/reports/graphs/LineGraph.tsx (+2 -2)
📝 packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx (+2 -2)
📝 packages/desktop-client/src/components/reports/graphs/adjustTextSize.ts (+24 -10)
packages/desktop-client/src/components/reports/graphs/tableGraph/RenderTableRow.tsx (+55 -0)
📝 packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTable.tsx (+26 -8)
📝 packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableHeader.tsx (+10 -10)
📝 packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableList.tsx (+52 -56)
📝 packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableRow.tsx (+2 -2)
📝 packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableTotals.tsx (+14 -9)
📝 packages/desktop-client/src/components/reports/reports/CustomReport.tsx (+3 -3)
📝 packages/desktop-client/src/components/reports/spreadsheets/calculateLegend.ts (+13 -10)
📝 packages/desktop-client/src/components/reports/spreadsheets/custom-spreadsheet.ts (+4 -7)
📝 packages/desktop-client/src/components/reports/spreadsheets/grouped-spreadsheet.ts (+3 -3)
📝 packages/loot-core/src/types/models/reports.d.ts (+12 -20)
upcoming-release-notes/2726.md (+6 -0)

📄 Description

Making files in custom reports to comply with TS strict - stage 1.


🔄 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/2726 **Author:** [@carkom](https://github.com/carkom) **Created:** 5/8/2024 **Status:** ✅ Merged **Merged:** 5/13/2024 **Merged by:** [@carkom](https://github.com/carkom) **Base:** `master` ← **Head:** `CR-strict1` --- ### 📝 Commits (5) - [`ef3ea83`](https://github.com/actualbudget/actual/commit/ef3ea83c132aeee15fb5386a7fbcbdc94f268a38) TS strict changes - [`7ae341b`](https://github.com/actualbudget/actual/commit/7ae341bffb55adc656b05c0e2bb30d251c228b30) notes - [`533d235`](https://github.com/actualbudget/actual/commit/533d235411af00d3b6835882d492e85956689373) renderRowProps - [`768ee2c`](https://github.com/actualbudget/actual/commit/768ee2cd6c9431d1cdd3ac9682d6935c2b261803) RenderTableRow - [`4a1c12c`](https://github.com/actualbudget/actual/commit/4a1c12c36de8f01e5d0d18d3f554c3108df148b0) Merge branch 'master' into CR-strict1 ### 📊 Changes **20 files changed** (+253 additions, -167 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/reports/ChooseGraph.tsx` (+4 -3) 📝 `packages/desktop-client/src/components/reports/ReportSummary.tsx` (+8 -9) 📝 `packages/desktop-client/src/components/reports/graphs/AreaGraph.tsx` (+3 -3) 📝 `packages/desktop-client/src/components/reports/graphs/BarGraph.tsx` (+7 -7) 📝 `packages/desktop-client/src/components/reports/graphs/DonutGraph.tsx` (+3 -3) 📝 `packages/desktop-client/src/components/reports/graphs/LineGraph.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/reports/graphs/adjustTextSize.ts` (+24 -10) ➕ `packages/desktop-client/src/components/reports/graphs/tableGraph/RenderTableRow.tsx` (+55 -0) 📝 `packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTable.tsx` (+26 -8) 📝 `packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableHeader.tsx` (+10 -10) 📝 `packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableList.tsx` (+52 -56) 📝 `packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableRow.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableTotals.tsx` (+14 -9) 📝 `packages/desktop-client/src/components/reports/reports/CustomReport.tsx` (+3 -3) 📝 `packages/desktop-client/src/components/reports/spreadsheets/calculateLegend.ts` (+13 -10) 📝 `packages/desktop-client/src/components/reports/spreadsheets/custom-spreadsheet.ts` (+4 -7) 📝 `packages/desktop-client/src/components/reports/spreadsheets/grouped-spreadsheet.ts` (+3 -3) 📝 `packages/loot-core/src/types/models/reports.d.ts` (+12 -20) ➕ `upcoming-release-notes/2726.md` (+6 -0) </details> ### 📄 Description Making files in custom reports to comply with TS strict - stage 1. --- <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-04-26 08:23:08 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#46296