[PR #4012] [MERGED] [Redux Toolkit Migration] accountsSlice #46971

Closed
opened 2026-04-26 08:59:20 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/4012
Author: @joel-jeremy
Created: 12/19/2024
Status: Merged
Merged: 1/13/2025
Merged by: @joel-jeremy

Base: masterHead: redux-toolkit-createSlice


📝 Commits (10+)

📊 Changes

34 files changed (+853 additions, -735 deletions)

View changed files

📝 packages/desktop-client/src/components/App.tsx (+15 -13)
📝 packages/desktop-client/src/components/accounts/Account.tsx (+79 -51)
📝 packages/desktop-client/src/components/accounts/AccountSyncCheck.tsx (+2 -2)
📝 packages/desktop-client/src/components/admin/UserAccess/UserAccessRow.tsx (+35 -28)
📝 packages/desktop-client/src/components/admin/UserDirectory/UserDirectory.tsx (+30 -20)
📝 packages/desktop-client/src/components/modals/EditAccess.tsx (+19 -16)
📝 packages/desktop-client/src/components/modals/EditUser.tsx (+22 -20)
📝 packages/desktop-client/src/components/modals/ImportTransactionsModal/ImportTransactionsModal.jsx (+3 -1)
📝 packages/desktop-client/src/components/modals/OpenIDEnableModal.tsx (+5 -5)
📝 packages/desktop-client/src/components/modals/PasswordEnableModal.tsx (+6 -5)
📝 packages/desktop-client/src/components/modals/SelectLinkedAccountsModal.jsx (+16 -14)
📝 packages/desktop-client/src/components/modals/TransferOwnership.tsx (+30 -21)
📝 packages/desktop-client/src/components/reports/reports/Calendar.tsx (+0 -1)
📝 packages/desktop-client/src/components/sidebar/Accounts.tsx (+3 -3)
📝 packages/desktop-client/src/components/transactions/TransactionList.jsx (+0 -2)
📝 packages/desktop-client/src/components/transactions/TransactionsTable.jsx (+8 -5)
📝 packages/desktop-client/src/global-events.ts (+61 -45)
packages/desktop-client/src/hooks/useActions.ts (+0 -33)
📝 packages/desktop-client/src/index.tsx (+10 -5)
packages/loot-core/src/client/accounts/accountsSlice.ts (+322 -0)

...and 14 more files

📄 Description

  • actions/account.ts / reducers/account.ts -> accounts/accountsSlice.ts [PR #1900] [MERGED] Larger mobile autocomplete fonts and paddings (#4012)
  • actions/queries.ts / reducers/queries.ts -> queries/queriesSlice.ts
  • actions/app.ts / reducers/app.ts -> app/appSlice.ts
  • actions/budgets.ts / reducers/budgets.ts -> budgets/budgetsSlice.ts
  • actions/modals.ts / reducers/modals.ts -> modals/modalsSlice.ts
  • actions/notifications.ts / reducers/notifications.ts -> notifications/notificationsSlice.ts
  • actions/prefs.ts / reducers/prefs.ts -> prefs/prefsSlice.ts
  • actions/user.ts / reducers/user.ts -> users/usersSlice.ts

🔄 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/4012 **Author:** [@joel-jeremy](https://github.com/joel-jeremy) **Created:** 12/19/2024 **Status:** ✅ Merged **Merged:** 1/13/2025 **Merged by:** [@joel-jeremy](https://github.com/joel-jeremy) **Base:** `master` ← **Head:** `redux-toolkit-createSlice` --- ### 📝 Commits (10+) - [`36a3c38`](https://github.com/actualbudget/actual/commit/36a3c38366651f04c7065bb792aaf0aec5cc8085) Migrate to accountSlice - [`67b1b5c`](https://github.com/actualbudget/actual/commit/67b1b5c2fc1996fb78c96c1437215b257336c231) Release notes - [`894a492`](https://github.com/actualbudget/actual/commit/894a4929a7deace6fa23db0ca2caccce9b5e045f) Fix lint and typecheck errors - [`27e9d26`](https://github.com/actualbudget/actual/commit/27e9d265e0d12a82f80bb699716032e48dd7e9a2) Update types - [`94a0e64`](https://github.com/actualbudget/actual/commit/94a0e6412176a405c0ed2ffb19f6dec01d05a377) Fix lint - [`fd89b22`](https://github.com/actualbudget/actual/commit/fd89b22dda78df579fa92cd19cef785551e7401c) Fix types - [`935d1ae`](https://github.com/actualbudget/actual/commit/935d1aee2ed3b42d5f8b83fac6d269f535d00308) Cleanup - [`fa3f587`](https://github.com/actualbudget/actual/commit/fa3f5874e9c746dcbe49a1c2487d1df39b64f2bc) Rename file - [`1bacc3b`](https://github.com/actualbudget/actual/commit/1bacc3b1f8fb8703d371f7e36892ade95a5f041b) Rename state - [`7191fa1`](https://github.com/actualbudget/actual/commit/7191fa1e7cd32674105c158df73813c4bfc28410) Cleanup types ### 📊 Changes **34 files changed** (+853 additions, -735 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/App.tsx` (+15 -13) 📝 `packages/desktop-client/src/components/accounts/Account.tsx` (+79 -51) 📝 `packages/desktop-client/src/components/accounts/AccountSyncCheck.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/admin/UserAccess/UserAccessRow.tsx` (+35 -28) 📝 `packages/desktop-client/src/components/admin/UserDirectory/UserDirectory.tsx` (+30 -20) 📝 `packages/desktop-client/src/components/modals/EditAccess.tsx` (+19 -16) 📝 `packages/desktop-client/src/components/modals/EditUser.tsx` (+22 -20) 📝 `packages/desktop-client/src/components/modals/ImportTransactionsModal/ImportTransactionsModal.jsx` (+3 -1) 📝 `packages/desktop-client/src/components/modals/OpenIDEnableModal.tsx` (+5 -5) 📝 `packages/desktop-client/src/components/modals/PasswordEnableModal.tsx` (+6 -5) 📝 `packages/desktop-client/src/components/modals/SelectLinkedAccountsModal.jsx` (+16 -14) 📝 `packages/desktop-client/src/components/modals/TransferOwnership.tsx` (+30 -21) 📝 `packages/desktop-client/src/components/reports/reports/Calendar.tsx` (+0 -1) 📝 `packages/desktop-client/src/components/sidebar/Accounts.tsx` (+3 -3) 📝 `packages/desktop-client/src/components/transactions/TransactionList.jsx` (+0 -2) 📝 `packages/desktop-client/src/components/transactions/TransactionsTable.jsx` (+8 -5) 📝 `packages/desktop-client/src/global-events.ts` (+61 -45) ➖ `packages/desktop-client/src/hooks/useActions.ts` (+0 -33) 📝 `packages/desktop-client/src/index.tsx` (+10 -5) ➕ `packages/loot-core/src/client/accounts/accountsSlice.ts` (+322 -0) _...and 14 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 - [ ] `actions/queries.ts` / `reducers/queries.ts` -> `queries/queriesSlice.ts` - [ ] `actions/app.ts` / `reducers/app.ts` -> `app/appSlice.ts` - [ ] `actions/budgets.ts` / `reducers/budgets.ts` -> `budgets/budgetsSlice.ts` - [ ] `actions/modals.ts` / `reducers/modals.ts` -> `modals/modalsSlice.ts` - [ ] `actions/notifications.ts` / `reducers/notifications.ts` -> `notifications/notificationsSlice.ts` - [ ] `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-04-26 08:59:20 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#46971