[PR #3365] [MERGED] refactor: fill in basic types in rules.ts #4816

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/3365
Author: @UnderKoen
Created: 9/4/2024
Status: Merged
Merged: 10/8/2024
Merged by: @MikesGlitch

Base: masterHead: typescript/simple-rules


📝 Commits (10+)

  • 36fabf3 refactor: fill in basic types in rules.ts
  • c0391a2 chore: release note
  • 8ddb8d4 Merge branch 'master' into typescript/simple-rules
  • 42e4c8d chore: implement pr feedback
  • 37607c6 Merge branch 'master' into typescript/simple-rules
  • 820b506 Merge branch 'master' into typescript/simple-rules
  • b4869bb Merge branch 'master' into typescript/simple-rules
  • 02827c4 Merge branch 'master' into typescript/simple-rules
  • bf73f06 Merge branch 'master' into typescript/simple-rules
  • 6f015ab Merge remote-tracking branch 'origin/master' into typescript/simple-rules

📊 Changes

4 files changed (+39 additions, -29 deletions)

View changed files

📝 packages/loot-core/src/server/accounts/rules.ts (+30 -26)
📝 packages/loot-core/src/server/accounts/transaction-rules.ts (+1 -1)
📝 packages/loot-core/src/types/models/rule.d.ts (+2 -2)
upcoming-release-notes/3365.md (+6 -0)

📄 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/3365 **Author:** [@UnderKoen](https://github.com/UnderKoen) **Created:** 9/4/2024 **Status:** ✅ Merged **Merged:** 10/8/2024 **Merged by:** [@MikesGlitch](https://github.com/MikesGlitch) **Base:** `master` ← **Head:** `typescript/simple-rules` --- ### 📝 Commits (10+) - [`36fabf3`](https://github.com/actualbudget/actual/commit/36fabf3a4dcf92c39313ebc94fde106abc87bc58) refactor: fill in basic types in rules.ts - [`c0391a2`](https://github.com/actualbudget/actual/commit/c0391a2d51a44678511e84e76cacb7fea913932b) chore: release note - [`8ddb8d4`](https://github.com/actualbudget/actual/commit/8ddb8d4ed0906d4ca3fe3ee6a10f61b91eb8935e) Merge branch 'master' into typescript/simple-rules - [`42e4c8d`](https://github.com/actualbudget/actual/commit/42e4c8d0803b00c89ffecdb20982fb1d0c1ac40a) chore: implement pr feedback - [`37607c6`](https://github.com/actualbudget/actual/commit/37607c67183904a11fb902110f179c3d39ec48e7) Merge branch 'master' into typescript/simple-rules - [`820b506`](https://github.com/actualbudget/actual/commit/820b506fa6572e836fe781388f3a76d085781f85) Merge branch 'master' into typescript/simple-rules - [`b4869bb`](https://github.com/actualbudget/actual/commit/b4869bb69f3eefcc536806f5c4e8ea876411067a) Merge branch 'master' into typescript/simple-rules - [`02827c4`](https://github.com/actualbudget/actual/commit/02827c4db25e3f75204e87f872ec48e8ea88a389) Merge branch 'master' into typescript/simple-rules - [`bf73f06`](https://github.com/actualbudget/actual/commit/bf73f064e783911e61528cba97c16658c95ca236) Merge branch 'master' into typescript/simple-rules - [`6f015ab`](https://github.com/actualbudget/actual/commit/6f015ab28a11dbed36e9dfb5fad064002c2bf180) Merge remote-tracking branch 'origin/master' into typescript/simple-rules ### 📊 Changes **4 files changed** (+39 additions, -29 deletions) <details> <summary>View changed files</summary> 📝 `packages/loot-core/src/server/accounts/rules.ts` (+30 -26) 📝 `packages/loot-core/src/server/accounts/transaction-rules.ts` (+1 -1) 📝 `packages/loot-core/src/types/models/rule.d.ts` (+2 -2) ➕ `upcoming-release-notes/3365.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 --> --- <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:01: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#4816