[PR #4127] [MERGED] [Redux Toolkit Migration] prefsSlice #5201

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/4127
Author: @joel-jeremy
Created: 1/10/2025
Status: Merged
Merged: 3/14/2025
Merged by: @joel-jeremy

Base: masterHead: redux-toolkit-prefs-slice


📝 Commits (10+)

📊 Changes

26 files changed (+236 additions, -224 deletions)

View changed files

📝 packages/desktop-client/src/browser-preload.browser.js (+1 -1)
📝 packages/desktop-client/src/components/App.tsx (+2 -1)
📝 packages/desktop-client/src/components/modals/CreateEncryptionKeyModal.tsx (+1 -1)
📝 packages/desktop-client/src/components/settings/index.tsx (+1 -1)
📝 packages/desktop-client/src/global-events.ts (+1 -1)
📝 packages/desktop-client/src/hooks/useGlobalPref.ts (+5 -5)
📝 packages/desktop-client/src/hooks/useMetadataPref.ts (+2 -2)
📝 packages/desktop-client/src/hooks/useSyncedPref.ts (+2 -2)
📝 packages/desktop-client/src/hooks/useSyncedPrefs.ts (+2 -2)
📝 packages/desktop-client/src/index.tsx (+2 -0)
📝 packages/desktop-electron/index.ts (+1 -1)
📝 packages/loot-core/src/client/actions/index.ts (+0 -1)
packages/loot-core/src/client/actions/prefs.ts (+0 -104)
📝 packages/loot-core/src/client/actions/user.ts (+1 -2)
📝 packages/loot-core/src/client/app/appSlice.ts (+1 -1)
📝 packages/loot-core/src/client/budgets/budgetsSlice.ts (+1 -1)
packages/loot-core/src/client/prefs/prefsSlice.ts (+170 -0)
📝 packages/loot-core/src/client/reducers/index.ts (+0 -2)
packages/loot-core/src/client/reducers/prefs.ts (+0 -39)
📝 packages/loot-core/src/client/shared-listeners.ts (+2 -1)

...and 6 more files

📄 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/4127 **Author:** [@joel-jeremy](https://github.com/joel-jeremy) **Created:** 1/10/2025 **Status:** ✅ Merged **Merged:** 3/14/2025 **Merged by:** [@joel-jeremy](https://github.com/joel-jeremy) **Base:** `master` ← **Head:** `redux-toolkit-prefs-slice` --- ### 📝 Commits (10+) - [`3994165`](https://github.com/actualbudget/actual/commit/39941652388c121cc32b2ff894dea27ab911ef68) Migrate to accountSlice - [`1e57013`](https://github.com/actualbudget/actual/commit/1e57013ad99c6da70941207255940999ae51cdc4) Fix lint and typecheck errors - [`ab20336`](https://github.com/actualbudget/actual/commit/ab203364699e844e3c7fc47b05760b3c8f8d2415) Update types - [`80e4d72`](https://github.com/actualbudget/actual/commit/80e4d7295efdf5ce78ca454b8137e9f76c42baf1) Fix lint - [`a518174`](https://github.com/actualbudget/actual/commit/a51817491dca92f796506969b0d538ed9f82f005) Fix types - [`e80bd16`](https://github.com/actualbudget/actual/commit/e80bd16d2eb0441e5585cd1625db585fb6c0c690) Cleanup - [`11c604a`](https://github.com/actualbudget/actual/commit/11c604ab01b712467a121dcb785caf2f3ebdf474) Rename file - [`b83d349`](https://github.com/actualbudget/actual/commit/b83d3493d0b032e531e6069deb76a1e20622b839) Rename state - [`2b95ff3`](https://github.com/actualbudget/actual/commit/2b95ff3485304703501453306629b0e5a644737f) Cleanup - [`59af8ff`](https://github.com/actualbudget/actual/commit/59af8ff9b92327a2766ac2355e10b5d6e3abccba) Fix typecheck error ### 📊 Changes **26 files changed** (+236 additions, -224 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/browser-preload.browser.js` (+1 -1) 📝 `packages/desktop-client/src/components/App.tsx` (+2 -1) 📝 `packages/desktop-client/src/components/modals/CreateEncryptionKeyModal.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/settings/index.tsx` (+1 -1) 📝 `packages/desktop-client/src/global-events.ts` (+1 -1) 📝 `packages/desktop-client/src/hooks/useGlobalPref.ts` (+5 -5) 📝 `packages/desktop-client/src/hooks/useMetadataPref.ts` (+2 -2) 📝 `packages/desktop-client/src/hooks/useSyncedPref.ts` (+2 -2) 📝 `packages/desktop-client/src/hooks/useSyncedPrefs.ts` (+2 -2) 📝 `packages/desktop-client/src/index.tsx` (+2 -0) 📝 `packages/desktop-electron/index.ts` (+1 -1) 📝 `packages/loot-core/src/client/actions/index.ts` (+0 -1) ➖ `packages/loot-core/src/client/actions/prefs.ts` (+0 -104) 📝 `packages/loot-core/src/client/actions/user.ts` (+1 -2) 📝 `packages/loot-core/src/client/app/appSlice.ts` (+1 -1) 📝 `packages/loot-core/src/client/budgets/budgetsSlice.ts` (+1 -1) ➕ `packages/loot-core/src/client/prefs/prefsSlice.ts` (+170 -0) 📝 `packages/loot-core/src/client/reducers/index.ts` (+0 -2) ➖ `packages/loot-core/src/client/reducers/prefs.ts` (+0 -39) 📝 `packages/loot-core/src/client/shared-listeners.ts` (+2 -1) _...and 6 more files_ </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 --> - [x] `actions/account.ts` / `reducers/account.ts` -> `accounts/accountsSlice.ts` #4012 - [x] `actions/queries.ts` / `reducers/queries.ts` -> `queries/queriesSlice.ts` #4016 - [x] `actions/app.ts` / `reducers/app.ts` -> `app/appSlice.ts` #4018 - [x] `actions/budgets.ts` / `reducers/budgets.ts` -> `budgets/budgetsSlice.ts` #4114 - [x] `actions/modals.ts` / `reducers/modals.ts` -> `modals/modalsSlice.ts` #4119 - [x] `actions/notifications.ts` / `reducers/notifications.ts` -> `notifications/notificationsSlice.ts` #4126 - [x] `actions/prefs.ts` / `reducers/prefs.ts` -> `prefs/prefsSlice.ts` - [ ] `actions/user.ts` / `reducers/user.ts` -> `users/usersSlice.ts` --- <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:58 -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#5201