[PR #4128] [MERGED] [Redux Toolkit Migration] usersSlice [Final PR] #5202

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

📋 Pull Request Information

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

Base: masterHead: redux-toolkit-users-slice


📝 Commits (10+)

📊 Changes

31 files changed (+184 additions, -194 deletions)

View changed files

📝 packages/desktop-client/src/components/App.tsx (+4 -2)
📝 packages/desktop-client/src/components/LoggedInUser.tsx (+1 -1)
📝 packages/desktop-client/src/components/admin/UserAccess/UserAccessRow.tsx (+1 -1)
📝 packages/desktop-client/src/components/admin/UserDirectory/UserDirectory.tsx (+4 -2)
📝 packages/desktop-client/src/components/manager/BudgetList.tsx (+1 -1)
📝 packages/desktop-client/src/components/manager/ConfigServer.tsx (+1 -1)
📝 packages/desktop-client/src/components/manager/ManagementApp.tsx (+1 -1)
📝 packages/desktop-client/src/components/manager/subscribe/Login.tsx (+1 -1)
📝 packages/desktop-client/src/components/manager/subscribe/OpenIdCallback.ts (+1 -1)
📝 packages/desktop-client/src/components/modals/EditAccess.tsx (+1 -1)
📝 packages/desktop-client/src/components/modals/EditUser.tsx (+1 -1)
📝 packages/desktop-client/src/index.tsx (+2 -2)
📝 packages/loot-core/src/client/accounts/accountsSlice.ts (+4 -0)
packages/loot-core/src/client/actions/index.ts (+0 -1)
packages/loot-core/src/client/actions/user.ts (+0 -50)
📝 packages/loot-core/src/client/app/appSlice.ts (+15 -1)
📝 packages/loot-core/src/client/budgets/budgetsSlice.ts (+10 -11)
packages/loot-core/src/client/constants.ts (+0 -21)
📝 packages/loot-core/src/client/modals/modalsSlice.ts (+7 -7)
📝 packages/loot-core/src/client/notifications/notificationsSlice.ts (+5 -0)

...and 11 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/4128 **Author:** [@joel-jeremy](https://github.com/joel-jeremy) **Created:** 1/10/2025 **Status:** ✅ Merged **Merged:** 3/17/2025 **Merged by:** [@joel-jeremy](https://github.com/joel-jeremy) **Base:** `master` ← **Head:** `redux-toolkit-users-slice` --- ### 📝 Commits (10+) - [`7b3fce9`](https://github.com/actualbudget/actual/commit/7b3fce956416fb80e69262bae561ec332345777b) Migrate to accountSlice - [`305943d`](https://github.com/actualbudget/actual/commit/305943dea4ab4f7c20abf1df3fe809930c49e85b) Fix lint and typecheck errors - [`548b8cd`](https://github.com/actualbudget/actual/commit/548b8cdde1292c80b5dc5247a5832aa641ef3b36) Update types - [`9a5f75b`](https://github.com/actualbudget/actual/commit/9a5f75bba3658b8b636b646a942a60ccbb5df637) Fix lint - [`1b8864e`](https://github.com/actualbudget/actual/commit/1b8864e067d38a83642a31887e2e06fdd1510b8c) Fix types - [`5bbd2e8`](https://github.com/actualbudget/actual/commit/5bbd2e8890ad62b2aa59d37530abc0d0e43ce3b2) Cleanup - [`75f66b7`](https://github.com/actualbudget/actual/commit/75f66b775c48f141bbe07be98dbe8f1f92130105) Rename file - [`ae4197a`](https://github.com/actualbudget/actual/commit/ae4197a1abd83086785a82b2acdbb4cf5c98c0d2) Rename state - [`309a741`](https://github.com/actualbudget/actual/commit/309a741a2134662305864f0cd77f3e8417075da5) Cleanup - [`bdde7a5`](https://github.com/actualbudget/actual/commit/bdde7a5be4a501ec5ed508e0871f881ad91eb05e) Fix typecheck error ### 📊 Changes **31 files changed** (+184 additions, -194 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/App.tsx` (+4 -2) 📝 `packages/desktop-client/src/components/LoggedInUser.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/admin/UserAccess/UserAccessRow.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/admin/UserDirectory/UserDirectory.tsx` (+4 -2) 📝 `packages/desktop-client/src/components/manager/BudgetList.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/manager/ConfigServer.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/manager/ManagementApp.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/manager/subscribe/Login.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/manager/subscribe/OpenIdCallback.ts` (+1 -1) 📝 `packages/desktop-client/src/components/modals/EditAccess.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/modals/EditUser.tsx` (+1 -1) 📝 `packages/desktop-client/src/index.tsx` (+2 -2) 📝 `packages/loot-core/src/client/accounts/accountsSlice.ts` (+4 -0) ➖ `packages/loot-core/src/client/actions/index.ts` (+0 -1) ➖ `packages/loot-core/src/client/actions/user.ts` (+0 -50) 📝 `packages/loot-core/src/client/app/appSlice.ts` (+15 -1) 📝 `packages/loot-core/src/client/budgets/budgetsSlice.ts` (+10 -11) ➖ `packages/loot-core/src/client/constants.ts` (+0 -21) 📝 `packages/loot-core/src/client/modals/modalsSlice.ts` (+7 -7) 📝 `packages/loot-core/src/client/notifications/notificationsSlice.ts` (+5 -0) _...and 11 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` #4127 - [x] `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:59 -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#5202