[PR #4249] [MERGED] [TypeScript] Make db.firstSync generic to make it easy to type DB query results #5283

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/4249
Author: @joel-jeremy
Created: 1/28/2025
Status: Merged
Merged: 3/4/2025
Merged by: @joel-jeremy

Base: masterHead: ts-db-firstSync


📝 Commits (10+)

📊 Changes

3 files changed (+38 additions, -26 deletions)

View changed files

📝 packages/loot-core/src/server/budget/actions.ts (+28 -19)
📝 packages/loot-core/src/server/db/index.ts (+4 -7)
upcoming-release-notes/4249.md (+6 -0)

📄 Description

Follow up to https://github.com/actualbudget/actual/pull/4247

  • db.runQuery
  • db.first
  • db.firstSync
  • db.all
  • db.select
  • db.selectWithSchema
  • db.selectFirstWithSchema

🔄 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/4249 **Author:** [@joel-jeremy](https://github.com/joel-jeremy) **Created:** 1/28/2025 **Status:** ✅ Merged **Merged:** 3/4/2025 **Merged by:** [@joel-jeremy](https://github.com/joel-jeremy) **Base:** `master` ← **Head:** `ts-db-firstSync` --- ### 📝 Commits (10+) - [`c0fde0d`](https://github.com/actualbudget/actual/commit/c0fde0d9d09cb73ac5ded0db51d8c88e08a118d9) [TypeScript] Make db.first generic - [`7425d0c`](https://github.com/actualbudget/actual/commit/7425d0cbd3c771d50d5a3be6b25882989f9f69bb) Release notes - [`9137c4f`](https://github.com/actualbudget/actual/commit/9137c4f432687e6d889a9bcedf6391b5a94ced42) Fix typecheck error - [`19fa629`](https://github.com/actualbudget/actual/commit/19fa6295856062f41aea9f80d2063ef8f4c7d875) Cleanup type - [`e06d75a`](https://github.com/actualbudget/actual/commit/e06d75a3974cfbc6ea3d6e37cfd3132573832471) Update db.first calls - [`553e00b`](https://github.com/actualbudget/actual/commit/553e00b4dfebf7c0427dff0423f8e6dfb0359ad0) Fix strict type - [`5b8c411`](https://github.com/actualbudget/actual/commit/5b8c4111347b8cd9cd4474683260b513d73f8cca) [TypeScript] Make db.firstSync generic - [`9a52b46`](https://github.com/actualbudget/actual/commit/9a52b461a25c0d14698e29a428a2dc2b6086adb5) Release notes - [`3a37adf`](https://github.com/actualbudget/actual/commit/3a37adfb2632492b54c85affd58e748c4bfe59d7) Merge branch 'master' into ts-db-firstSync - [`125f854`](https://github.com/actualbudget/actual/commit/125f8542c7e3d6224ba2a702af2df25fa5b54113) Coderabbit review ### 📊 Changes **3 files changed** (+38 additions, -26 deletions) <details> <summary>View changed files</summary> 📝 `packages/loot-core/src/server/budget/actions.ts` (+28 -19) 📝 `packages/loot-core/src/server/db/index.ts` (+4 -7) ➕ `upcoming-release-notes/4249.md` (+6 -0) </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 --> Follow up to https://github.com/actualbudget/actual/pull/4247 - [x] `db.runQuery` - [x] `db.first` - [x] `db.firstSync` - [ ] `db.all` - [ ] `db.select` - [ ] `db.selectWithSchema` - [ ] `db.selectFirstWithSchema` --- <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:09: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#5283