[PR #3236] [MERGED] ♻️ (prefs) initial type implementation for synced/local/metadata prefs #4749

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/3236
Author: @MatissJanis
Created: 8/11/2024
Status: Merged
Merged: 8/12/2024
Merged by: @MatissJanis

Base: masterHead: matiss/synced-prefs-2


📝 Commits (2)

  • 04c621d ♻️ (prefs) initial type implementation for synced/local/metadata prefs
  • 6af38dd Patch linter

📊 Changes

43 files changed (+196 additions, -126 deletions)

View changed files

📝 packages/desktop-client/src/components/App.tsx (+3 -3)
📝 packages/desktop-client/src/components/Titlebar.tsx (+4 -3)
📝 packages/desktop-client/src/components/accounts/Account.jsx (+7 -7)
📝 packages/desktop-client/src/components/autocomplete/CategoryAutocomplete.tsx (+2 -2)
📝 packages/desktop-client/src/components/budget/index.tsx (+2 -1)
📝 packages/desktop-client/src/components/budget/util.ts (+4 -4)
📝 packages/desktop-client/src/components/manager/BudgetList.tsx (+2 -2)
📝 packages/desktop-client/src/components/mobile/accounts/Account.jsx (+3 -3)
📝 packages/desktop-client/src/components/mobile/accounts/AccountTransactions.jsx (+2 -2)
📝 packages/desktop-client/src/components/mobile/accounts/Accounts.jsx (+3 -3)
📝 packages/desktop-client/src/components/mobile/budget/BudgetTable.jsx (+3 -2)
📝 packages/desktop-client/src/components/mobile/budget/Category.tsx (+3 -3)
📝 packages/desktop-client/src/components/mobile/budget/CategoryTransactions.jsx (+2 -2)
📝 packages/desktop-client/src/components/mobile/budget/index.tsx (+4 -3)
📝 packages/desktop-client/src/components/mobile/transactions/FocusableAmountInput.tsx (+2 -2)
📝 packages/desktop-client/src/components/modals/BudgetListModal.tsx (+2 -2)
📝 packages/desktop-client/src/components/modals/LoadBackup.jsx (+2 -2)
📝 packages/desktop-client/src/components/reports/ReportSidebar.tsx (+2 -2)
📝 packages/desktop-client/src/components/reports/getLiveRange.ts (+2 -2)
📝 packages/desktop-client/src/components/reports/reportRanges.ts (+5 -5)

...and 23 more files

📄 Description

Overall objective: make preferences easier to reason about; introduce a concept of "synced" (cross-device) preferences. More info: https://github.com/actualbudget/actual/pull/3087

This PR does not have functional changes. I am only laying the groundwork for the synced prefs/metadata prefs/local storage prefs.

Creating three types and three hooks for retrieving the preferences. The storage is still the same for all of these. Real logic will come in follow-up PRs so as to make the code-review scope small and to reduce the risk of breaking things.


🔄 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/3236 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 8/11/2024 **Status:** ✅ Merged **Merged:** 8/12/2024 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/synced-prefs-2` --- ### 📝 Commits (2) - [`04c621d`](https://github.com/actualbudget/actual/commit/04c621de8dff284ad3b701d5f76f4766d5af516e) :recycle: (prefs) initial type implementation for synced/local/metadata prefs - [`6af38dd`](https://github.com/actualbudget/actual/commit/6af38dd42cbc0644f1d2cb85b097079f0e19597d) Patch linter ### 📊 Changes **43 files changed** (+196 additions, -126 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/App.tsx` (+3 -3) 📝 `packages/desktop-client/src/components/Titlebar.tsx` (+4 -3) 📝 `packages/desktop-client/src/components/accounts/Account.jsx` (+7 -7) 📝 `packages/desktop-client/src/components/autocomplete/CategoryAutocomplete.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/budget/index.tsx` (+2 -1) 📝 `packages/desktop-client/src/components/budget/util.ts` (+4 -4) 📝 `packages/desktop-client/src/components/manager/BudgetList.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/mobile/accounts/Account.jsx` (+3 -3) 📝 `packages/desktop-client/src/components/mobile/accounts/AccountTransactions.jsx` (+2 -2) 📝 `packages/desktop-client/src/components/mobile/accounts/Accounts.jsx` (+3 -3) 📝 `packages/desktop-client/src/components/mobile/budget/BudgetTable.jsx` (+3 -2) 📝 `packages/desktop-client/src/components/mobile/budget/Category.tsx` (+3 -3) 📝 `packages/desktop-client/src/components/mobile/budget/CategoryTransactions.jsx` (+2 -2) 📝 `packages/desktop-client/src/components/mobile/budget/index.tsx` (+4 -3) 📝 `packages/desktop-client/src/components/mobile/transactions/FocusableAmountInput.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/modals/BudgetListModal.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/modals/LoadBackup.jsx` (+2 -2) 📝 `packages/desktop-client/src/components/reports/ReportSidebar.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/reports/getLiveRange.ts` (+2 -2) 📝 `packages/desktop-client/src/components/reports/reportRanges.ts` (+5 -5) _...and 23 more files_ </details> ### 📄 Description Overall objective: make preferences easier to reason about; introduce a concept of "synced" (cross-device) preferences. More info: https://github.com/actualbudget/actual/pull/3087 This PR does not have functional changes. I am only laying the groundwork for the synced prefs/metadata prefs/local storage prefs. Creating three types and three hooks for retrieving the preferences. The storage is still the same for all of these. Real logic will come in follow-up PRs so as to make the code-review scope small and to reduce the risk of breaking things. --- <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:00:40 -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#4749