[PR #4126] [MERGED] [Redux Toolkit Migration] notificationsSlice #5200

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

📋 Pull Request Information

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

Base: masterHead: redux-toolkit-notifications-slice


📝 Commits (10+)

📊 Changes

45 files changed (+549 additions, -444 deletions)

View changed files

📝 packages/desktop-client/src/components/App.tsx (+12 -13)
📝 packages/desktop-client/src/components/FinancesApp.tsx (+31 -25)
📝 packages/desktop-client/src/components/Notifications.tsx (+5 -3)
📝 packages/desktop-client/src/components/ServerContext.tsx (+14 -7)
📝 packages/desktop-client/src/components/accounts/Account.tsx (+5 -3)
📝 packages/desktop-client/src/components/admin/UserAccess/UserAccess.tsx (+14 -10)
📝 packages/desktop-client/src/components/admin/UserAccess/UserAccessRow.tsx (+27 -20)
📝 packages/desktop-client/src/components/admin/UserDirectory/UserDirectory.tsx (+25 -18)
📝 packages/desktop-client/src/components/budget/index.tsx (+8 -6)
📝 packages/desktop-client/src/components/manager/subscribe/OpenIdForm.tsx (+15 -10)
📝 packages/desktop-client/src/components/mobile/transactions/TransactionList.tsx (+4 -2)
📝 packages/desktop-client/src/components/modals/CreateAccountModal.tsx (+7 -5)
📝 packages/desktop-client/src/components/modals/EditAccess.tsx (+13 -10)
📝 packages/desktop-client/src/components/modals/EditUser.tsx (+13 -10)
📝 packages/desktop-client/src/components/modals/TransferOwnership.tsx (+17 -13)
📝 packages/desktop-client/src/components/modals/manager/ConfirmChangeDocumentDir.tsx (+5 -3)
📝 packages/desktop-client/src/components/modals/manager/DuplicateFileModal.tsx (+9 -5)
📝 packages/desktop-client/src/components/reports/Overview.tsx (+37 -24)
📝 packages/desktop-client/src/components/reports/reports/Calendar.tsx (+9 -5)
📝 packages/desktop-client/src/components/reports/reports/CashFlow.tsx (+5 -3)

...and 25 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/4126 **Author:** [@joel-jeremy](https://github.com/joel-jeremy) **Created:** 1/10/2025 **Status:** ✅ Merged **Merged:** 3/7/2025 **Merged by:** [@joel-jeremy](https://github.com/joel-jeremy) **Base:** `master` ← **Head:** `redux-toolkit-notifications-slice` --- ### 📝 Commits (10+) - [`ea2019f`](https://github.com/actualbudget/actual/commit/ea2019fea5ab7a9a11d46f902228bd64295d16ef) Migrate to accountSlice - [`87cb397`](https://github.com/actualbudget/actual/commit/87cb3979889f1cf949f50a97af8dbafbc0ff6d57) Fix lint and typecheck errors - [`9209674`](https://github.com/actualbudget/actual/commit/920967496bad4e30724d09b782c6085ffef7558f) Update types - [`3701001`](https://github.com/actualbudget/actual/commit/37010015d18af64c1a8afa2631bcc4277691b999) Fix lint - [`f0a992a`](https://github.com/actualbudget/actual/commit/f0a992a4d4765fee498e017237fb07fda6e4d7b8) Fix types - [`ab42d7d`](https://github.com/actualbudget/actual/commit/ab42d7d5be95188d7a383aee8e982c8b87427329) Cleanup - [`afffad1`](https://github.com/actualbudget/actual/commit/afffad17ed83b3a170774d1d7107b3b22e8819ef) Rename file - [`ce58fa6`](https://github.com/actualbudget/actual/commit/ce58fa6f687944e9071dca8eb64235e11823da7c) Rename state - [`94a3a38`](https://github.com/actualbudget/actual/commit/94a3a38ffa69ab8841c7a08255ebded871b43d22) Cleanup - [`3a7cf27`](https://github.com/actualbudget/actual/commit/3a7cf279275ec6eb9811ca43616d49a36ac5465b) Fix typecheck error ### 📊 Changes **45 files changed** (+549 additions, -444 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/App.tsx` (+12 -13) 📝 `packages/desktop-client/src/components/FinancesApp.tsx` (+31 -25) 📝 `packages/desktop-client/src/components/Notifications.tsx` (+5 -3) 📝 `packages/desktop-client/src/components/ServerContext.tsx` (+14 -7) 📝 `packages/desktop-client/src/components/accounts/Account.tsx` (+5 -3) 📝 `packages/desktop-client/src/components/admin/UserAccess/UserAccess.tsx` (+14 -10) 📝 `packages/desktop-client/src/components/admin/UserAccess/UserAccessRow.tsx` (+27 -20) 📝 `packages/desktop-client/src/components/admin/UserDirectory/UserDirectory.tsx` (+25 -18) 📝 `packages/desktop-client/src/components/budget/index.tsx` (+8 -6) 📝 `packages/desktop-client/src/components/manager/subscribe/OpenIdForm.tsx` (+15 -10) 📝 `packages/desktop-client/src/components/mobile/transactions/TransactionList.tsx` (+4 -2) 📝 `packages/desktop-client/src/components/modals/CreateAccountModal.tsx` (+7 -5) 📝 `packages/desktop-client/src/components/modals/EditAccess.tsx` (+13 -10) 📝 `packages/desktop-client/src/components/modals/EditUser.tsx` (+13 -10) 📝 `packages/desktop-client/src/components/modals/TransferOwnership.tsx` (+17 -13) 📝 `packages/desktop-client/src/components/modals/manager/ConfirmChangeDocumentDir.tsx` (+5 -3) 📝 `packages/desktop-client/src/components/modals/manager/DuplicateFileModal.tsx` (+9 -5) 📝 `packages/desktop-client/src/components/reports/Overview.tsx` (+37 -24) 📝 `packages/desktop-client/src/components/reports/reports/Calendar.tsx` (+9 -5) 📝 `packages/desktop-client/src/components/reports/reports/CashFlow.tsx` (+5 -3) _...and 25 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` - [ ] `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:56 -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#5200