[PR #4108] [MERGED] ♻️ (typescript) ported TransactionsTable.test to TS #5190

Closed
opened 2026-02-28 21:07:42 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/4108
Author: @MatissJanis
Created: 1/7/2025
Status: Merged
Merged: 1/14/2025
Merged by: @MatissJanis

Base: masterHead: matiss/typescript-trans-test


📝 Commits (3)

  • 5269d52 ♻️ (typescript) ported TransactionsTable.test to TS
  • 1ed938d Patch lint & test
  • 33fabe6 Merge branch 'master' into matiss/typescript-trans-test

📊 Changes

14 files changed (+225 additions, -246 deletions)

View changed files

📝 packages/desktop-client/globals.d.ts (+4 -0)
📝 packages/desktop-client/package.json (+0 -2)
📝 packages/desktop-client/src/components/transactions/TransactionsTable.test.tsx (+107 -63)
📝 packages/desktop-client/src/hooks/useSplitsExpanded.tsx (+1 -1)
📝 packages/loot-core/src/client/query-helpers.test.ts (+3 -1)
📝 packages/loot-core/src/mocks/index.ts (+65 -33)
📝 packages/loot-core/src/platform/client/fetch/index.d.ts (+9 -1)
📝 packages/loot-core/src/server/accounts/rules.test.ts (+0 -6)
📝 packages/loot-core/src/server/spreadsheet/spreadsheet.test.ts (+3 -6)
📝 packages/loot-core/src/shared/transactions.ts (+19 -20)
📝 packages/loot-core/src/types/models/transaction.d.ts (+5 -0)
📝 tsconfig.json (+1 -1)
upcoming-release-notes/4108.md (+6 -0)
📝 yarn.lock (+2 -112)

📄 Description


🔄 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/4108 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 1/7/2025 **Status:** ✅ Merged **Merged:** 1/14/2025 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/typescript-trans-test` --- ### 📝 Commits (3) - [`5269d52`](https://github.com/actualbudget/actual/commit/5269d52cffb946cabe3996e74bd6d55ee153336f) :recycle: (typescript) ported TransactionsTable.test to TS - [`1ed938d`](https://github.com/actualbudget/actual/commit/1ed938d7819d1d5c280ece20132757f5de7ec6c3) Patch lint & test - [`33fabe6`](https://github.com/actualbudget/actual/commit/33fabe69b73a1471a4bcb366b89a04c76a139abf) Merge branch 'master' into matiss/typescript-trans-test ### 📊 Changes **14 files changed** (+225 additions, -246 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/globals.d.ts` (+4 -0) 📝 `packages/desktop-client/package.json` (+0 -2) 📝 `packages/desktop-client/src/components/transactions/TransactionsTable.test.tsx` (+107 -63) 📝 `packages/desktop-client/src/hooks/useSplitsExpanded.tsx` (+1 -1) 📝 `packages/loot-core/src/client/query-helpers.test.ts` (+3 -1) 📝 `packages/loot-core/src/mocks/index.ts` (+65 -33) 📝 `packages/loot-core/src/platform/client/fetch/index.d.ts` (+9 -1) 📝 `packages/loot-core/src/server/accounts/rules.test.ts` (+0 -6) 📝 `packages/loot-core/src/server/spreadsheet/spreadsheet.test.ts` (+3 -6) 📝 `packages/loot-core/src/shared/transactions.ts` (+19 -20) 📝 `packages/loot-core/src/types/models/transaction.d.ts` (+5 -0) 📝 `tsconfig.json` (+1 -1) ➕ `upcoming-release-notes/4108.md` (+6 -0) 📝 `yarn.lock` (+2 -112) </details> ### 📄 Description <!-- Thank you for submitting a pull request! Make sure to follow the instructions to write release notes for your PR — it should only take a minute or two: https://github.com/actualbudget/docs#writing-good-release-notes --> --- <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 21:07:42 -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#5190