mirror of
https://github.com/actualbudget/actual.git
synced 2026-07-16 23:26:48 -05:00
[BUG] Auto categorization rule not created on new transaction or update of transaction #124
Closed
opened 2026-02-28 18:49:24 -06:00 by GiteaMirror
·
6 comments
No Branch/Tag Specified
master
sqlparam-nullable-query-params
cross-version-sync-stage2
claude/transaction-column-manager-cp0u0w
claude/verify-migration-guardrails-i82c8m
dependabot/npm_and_yarn/npm_and_yarn-cd62b4b37a
tabedzki/feat/replacing-month-picker-with-day-picker
delay-initial-sync-until-loaded
jfdoming/mobile-app-build
cross-version-sync-compat
jfdoming/api-tokens-part-1
jfdoming/api-tokens-part-2
jfdoming/api-tokens-part-3
jfdoming/mobile-app-updates
claude/budget-version-migration-compat-151jlr
claude/worker-error-serialization-9iqtl7
claude/react-table-library-research-afx83z
claude/issue-8401-regression-test-9nw9l3
jfdoming/mobile-app-foundation-part-1
jfdoming/mobile-app-scaffold
release
ai-fix-cleanup-def-8290
claude/relaxed-shannon-jusywa
claude/fervent-keller-s6f2fq
claude/determined-feynman-st1y9z
claude/wizardly-einstein-vi2wce
claude/sweet-albattani-7nseio
feat/bank-sync-per-budget-file
claude/affectionate-cannon-y0kc82
claude/dependabot-findings-m9pgmb
claude/mac-app-ios-publish-zjs1lz
claude/vigilant-tesla-aiiyos
depot-migrate-secrets-1781296330308
claude/quirky-cerf-v5qck7
claude/animation-removal-alternative-2etfh1
claude/netlify-preview-comments-consolidate-9XPYa
osc/7391-error-boundaries-reports
claude/pensive-dijkstra-7DYui
claude/server-app-version-mismatch-HTCGv
claude/pr-8027-review-feedback-mELuO
claude/zealous-wozniak-e5pO0
claude/intelligent-heisenberg-BSHJq
claude/translation-download-refactor-jFRsO
claude/mobile-performance-ideas-G9PAO
claude/github-issue-7988-5A5ly
blacksmith-migration-ff70d2d
claude/great-mayer-KRvOk
claude/react-native-ios-compilation-fcJh5
claude/serene-goldberg-58xv0
claude/kind-euler-LVG3K
release-automation/edge-nightly
7710-bundle-migrations
claude/fix-docker-build-error-WMfed
claude/llm-compatible-release-notes-t20Nr
7710-tests-tdd
cursor/transaction-table-rewrite-f077
claude/plan-ci-secure-context-OtEe1
claude/fix-issue-7667-DPXi3
cursor/resolve-pr-7449-ee11
claude/fix-typescript-build-error-JPtZ5
copilot/add-repository-configs-to-packages
worktree-compressed-drifting-ritchie
claude/api-consumer-verification-kfz1K
pr-7454
claude/fix-issue-7410-LLLQ4
revert-7281-generate-icons
react-query-useSchedules
copilot/sub-pr-6140
package-upgrades
v26.7.0
v26.6.0
v26.5.2
v26.5.1
v26.5.0
v26.4.0
v26.3.0
v26.2.1
v26.2.0
v26.1.0
v25.12.0
v25.11.0
v25.10.0
v25.9.0
v25.8.0
v25.7.1
v25.7.0
v25.6.1
v25.6.0
v25.5.0
v25.4.0
v25.3.1
v25.3.0
v25.2.1
v25.2.0
v25.1.0
v24.12.0
v24.11.0
v24.10.1
v24.10.0
v24.9.0
v24.8.0
v24.7.0
v24.6.0
v24.5.0
v24.4.0
v24.3.0
v24.2.0
v24.1.0
v23.12.0
v23.11.0
v23.10.0
v23.9.0
v23.8.1
v23.8.0
v23.7.2
v23.7.1
v23.7.0
v23.6.0
v23.5.0
v23.4.2
v23.4.1
v23.4.0
v23.3.2
v23.3.0
v23.2.9
v23.2.5
v23.1.12
v22.12.9
Labels
Clear labels
AI generated
API
bank sync
budgeting
bug
can’t replicate
dependencies
docker
documentation
electron
experimental feature
feature
feedback
goal templates
good first issue
help wanted
importers
maintenance
needs info
needs testing
needs triage
needs votes
openid
payees
pull-request
regression
reports
responsive
rules
schedules
server
✨ merged
split transactions
tech debt
tech-support
theme
transaction import
transaction reconciliation
transactions
translations
upstream
user interface
✅ approved
wontfix
Mirrored from GitHub Pull Request
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/actual#124
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @winklevos on GitHub (Aug 21, 2022).
When creating a new transaction and a new payee, adding a category does not auto-create a related rule
The same occurs when updating an existing transaction from uncategorised
Browser: Firefox and Chrome
Version: https://hub.docker.com/r/kippenhof/actual-server latest
Confirmed the same behaviour locally building from master
@rich-howell commented on GitHub (Aug 21, 2022):
Hey @winklevos
This looks to be by design, if you take a look at the documentation https://actualbudget.com/docs/other/rules/ it states the following;
Actual will create a rule based on how you use it.
Hope this helps, if you need anything more please do reach out or @ me in discord and I will re-open this issue.
@winklevos commented on GitHub (Aug 21, 2022):
@rich-howell this bug is saying it isn't creating as per this documentation.
When categorising transactions rules are not created automatically.
I have a working fix in progress that adds the behaviour I expect
@rich-howell commented on GitHub (Aug 22, 2022):
I am happy to re-open this for you, however this isn't a bug.
The rules engine will decide when a rule needs to be created based on the factors used within the transaction. If you just create one transaction for say Grocery and select the merchant and value a rule won't be created because you would end up with rules for every single thing.
I am not sure if you will get your PR merged into main if it create a rule for every single transaction added, but I will leave this open for James to review when the time comes.
@winklevos commented on GitHub (Aug 22, 2022):
This is the exact user experience I expect and is standard in budgeting apps. Ultimately a user should not have to manually categorise as much as possible.
If creating a rule for each payee is going to be a problem we might need to change some things.
I see the functionality like so
I've imported 10,000 transactions into actual, the user experience currently will result in me not being able to use it.
I hope this feature can be considered beneficial for actual
@winklevos commented on GitHub (Aug 23, 2022):
For note, the current logic is in https://github.com/winklevos/actual/blob/categorise-rules/packages/loot-core/src/server/accounts/transaction-rules.js#L614
Goes as follows
When the category is changed actual will look back 6 months, and forward 180 days. And get all related transactions for that payee.
It will only continue if you modified the latest transactions.
It will then count the number of occurrences of that category for the payee.
It selects the category with the majority (max). And only updates if it has > 3 occurrences.
Think
This to me is a bit overbuilt, see my previous three points for UX
@rich-howell commented on GitHub (Sep 7, 2022):
As this isn't a bug I am going to move it over to ideas for further discussion.