🔖 (24.10.0) #3545
Merged
🔖 (24.10.0) #3545
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
MatissJanis
commented
Oct 2, 2024
•
MatissJanis
commented
- web: 🔖 (24.10.0) #3545
- server: 🔖 (24.10.0) actual-server#472
- docs: 🔖 (24.10.0) docs#460
WalkthroughThe pull request includes updates to the Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
✅ Deploy Preview for actualbudget ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Here are the automatically generated release notes! Version: v24.10.0
#### Features
- [#3325](https://github.com/actualbudget/actual/pull/3325) Add setting to set preferred dark theme for "System default" mode — thanks @tim-smart
- [#3326](https://github.com/actualbudget/actual/pull/3326) Add an uncategorized transaction button to the mobile app. — thanks @tim-smart
- [#3459](https://github.com/actualbudget/actual/pull/3459) SimpleFIN is officially released as a first-party feature of Actual. — thanks @MatissJanis
#### Enhancements
- [#3085](https://github.com/actualbudget/actual/pull/3085) Show undo notifications when applying goal templates / budget actions in mobile. — thanks @joel-jeremy
- [#3119](https://github.com/actualbudget/actual/pull/3119) Auto notes in month notes when reassigning budgets. — thanks @joel-jeremy
- [#3288](https://github.com/actualbudget/actual/pull/3288) Dashboards: text widget support. — thanks @Matissjanis
- [#3298](https://github.com/actualbudget/actual/pull/3298) Implement greater visibility and accessibility on popover menus and tooltips. — thanks @VoltaicGRiD & @psybers
- [#3313](https://github.com/actualbudget/actual/pull/3313) Support translations in desktop-client/components/schedules. — thanks @psybers
- [#3332](https://github.com/actualbudget/actual/pull/3332) Set html theme-color meta tag to match the webmanifest theme_color. — thanks @tim-smart
- [#3344](https://github.com/actualbudget/actual/pull/3344) Adjust mobile header link style — thanks @tim-smart
- [#3364](https://github.com/actualbudget/actual/pull/3364) Dashboards: ability to save filters & time-range on net-worth widgets. — thanks @MatissJanis & @carkom
- [#3377](https://github.com/actualbudget/actual/pull/3377) Privacy mode: instead of blurring, use an illegible font (#3376) — thanks @olets
- [#3380](https://github.com/actualbudget/actual/pull/3380) Fixing spending report header so that any month can be compared to any other month. Also adds budget as an optional comparison. — thanks @carkom
- [#3400](https://github.com/actualbudget/actual/pull/3400) Update splash background to match theming — thanks @jfdoming
- [#3411](https://github.com/actualbudget/actual/pull/3411) Undoable auto tranfer notes + auto notes for cover — thanks @joel-jeremy
- [#3417](https://github.com/actualbudget/actual/pull/3417) Dashboards: ability to save filters & time-range on cash-flow widgets. — thanks @MatissJanis
- [#3432](https://github.com/actualbudget/actual/pull/3432) Dashboards: ability to save filters & time-range on spending widgets. — thanks @MatissJanis
- [#3451](https://github.com/actualbudget/actual/pull/3451) Fixing display issues in spending report and adding a legend. — thanks @carkom
#### Bugfix
- [#3286](https://github.com/actualbudget/actual/pull/3286) Forcibly reload app when API request is redirected. This fixes issue #2793 — thanks @TimQuelch
- [#3356](https://github.com/actualbudget/actual/pull/3356) Fix incorrect hook usage in #3277 — thanks @matt-fidd
- [#3374](https://github.com/actualbudget/actual/pull/3374) Mobile - Fix budget cells being triggered when pulling down to refresh on budget table. — thanks @joel-jeremy
- [#3379](https://github.com/actualbudget/actual/pull/3379) Fix report header validateStart bug. — thanks @carkom
- [#3389](https://github.com/actualbudget/actual/pull/3389) Prevent keyboard shortcuts modal launching when entering shortcut into editable field. — thanks @MikesGlitch
- [#3399](https://github.com/actualbudget/actual/pull/3399) Fix regression in case sensitivity for `is`/`matches` operator — thanks @jfdoming
- [#3412](https://github.com/actualbudget/actual/pull/3412) Sort suggested payee popup section by favorite status first, then alphabetically. — thanks @qedi-r
- [#3441](https://github.com/actualbudget/actual/pull/3441) Fix account filters being overridden. — thanks @MatisJanis
- [#3453](https://github.com/actualbudget/actual/pull/3453) Fix react-aria-components Button hover styles — thanks @joel-jeremy
- [#3456](https://github.com/actualbudget/actual/pull/3456) Fix csv import deduplication #3416 — thanks @UnderKoen
- [#3460](https://github.com/actualbudget/actual/pull/3460) SyncedPrefs: fix import prefs not reading correctly. — thanks @MatisJanis
- [#3462](https://github.com/actualbudget/actual/pull/3462) Fix save report forms submit handler so that it doesn't trigger a reload of an entire page on submit. — thanks @joel-jeremy
- [#3463](https://github.com/actualbudget/actual/pull/3463) Mobile - Do not allow renaming category or group to an empty name. — thanks @joel-jeremy
- [#3469](https://github.com/actualbudget/actual/pull/3469) Reports - deleting custom reports should remove the widget from the dashboard. — thanks @MatissJanis
- [#3472](https://github.com/actualbudget/actual/pull/3472) Fix privacy filters not activating — thanks @joel-jeremy
- [#3478](https://github.com/actualbudget/actual/pull/3478) CSV import - fix checkboxes not working. — thanks @MatissJanis
- [#3480](https://github.com/actualbudget/actual/pull/3480) Fix 'matches' operator incorrectly matching empty strings. — thanks @qedi-r
- [#3484](https://github.com/actualbudget/actual/pull/3484) Custom reports - fix opening table reports crashing the page. — thanks @MatissJanis
- [#3487](https://github.com/actualbudget/actual/pull/3487) Fix modals not opening on mobile budget view — thanks @tim-smart
- [#3491](https://github.com/actualbudget/actual/pull/3491) Fix mobile page header button styles. — thanks @joel-jeremy
- [#3492](https://github.com/actualbudget/actual/pull/3492) Fix mobile balance modal not properly coloring balance. — thanks @joel-jeremy
- [#3495](https://github.com/actualbudget/actual/pull/3495) Removes whitespace from both ends of the category name — thanks @junyuanz1
- [#3501](https://github.com/actualbudget/actual/pull/3501) Fix typo in README for Actual Budget Github homepage — thanks @Jonathan-Fang
- [#3509](https://github.com/actualbudget/actual/pull/3509) Fix desktop popover showing in mobile To Budget options. — thanks @qedi-r
- [#3510](https://github.com/actualbudget/actual/pull/3510) Fix templates and syntax check running on deleted categories — thanks @youngcw
- [#3515](https://github.com/actualbudget/actual/pull/3515) Fix GoCardless linking (account selection window not appearing) — thanks @EtaoinWu
- [#3535](https://github.com/actualbudget/actual/pull/3535) Fix balance carryover arrow on Firefox — thanks @MikesGlitch
- [#3539](https://github.com/actualbudget/actual/pull/3539) Fix error when creating a rule from an existing transaction — thanks @matt-fidd
- [#3540](https://github.com/actualbudget/actual/pull/3540) BankSync: fix showing sync status after linking and close all modal stack. — thanks @MatissJanis
- [#3541](https://github.com/actualbudget/actual/pull/3541) Fix import crashing if the QFX file is malformed. — thanks @MatissJanis
#### Maintenance
- [#3163](https://github.com/actualbudget/actual/pull/3163) Use new react-aria-components in common and mobile components. — thanks @joel-jeremy
- [#3277](https://github.com/actualbudget/actual/pull/3277) Support translations in desktop-client/components/accounts. — thanks @matt-fidd
- [#3311](https://github.com/actualbudget/actual/pull/3311) TypeScript: migrate Account component. — thanks @MatissJanis
- [#3352](https://github.com/actualbudget/actual/pull/3352) Support translations in Translation support for desktop-client/src/components/budget/BalanceWithCarryover.tsx — thanks @nmathey
- [#3355](https://github.com/actualbudget/actual/pull/3355) TypeScript: migrate Reconcile file. — thanks @MatissJanis
- [#3361](https://github.com/actualbudget/actual/pull/3361) TypeScript: RSchedule types fixed — thanks @UnderKoen
- [#3363](https://github.com/actualbudget/actual/pull/3363) Support translations in desktop-client/components/budget. — thanks @agradina
- [#3367](https://github.com/actualbudget/actual/pull/3367) Support translations in Translation support for desktop-client/src/components/budget/BudgetTotals.tsx — thanks @nmathey
- [#3391](https://github.com/actualbudget/actual/pull/3391) Refine Menu/Select types to allow broader types for the value/name attribute — thanks @jfdoming
- [#3393](https://github.com/actualbudget/actual/pull/3393) Prevent sidebar from saving to metadata.json unnecessarily — thanks @MikesGlitch
- [#3394](https://github.com/actualbudget/actual/pull/3394) Remove unneded Payee typeahead console log — thanks @MikesGlitch
- [#3395](https://github.com/actualbudget/actual/pull/3395) SyncedPrefs: refactor pref values from explicit types to `string`. — thanks @MatissJanis
- [#3396](https://github.com/actualbudget/actual/pull/3396) Convert `RecurringSchedulePicker.jsx` -> `RecurringSchedulePicker.tsx` — thanks @jfdoming
- [#3397](https://github.com/actualbudget/actual/pull/3397) SyncedPrefs: refactor usages of number formatter away from redux. — thanks @MatissJanis
- [#3398](https://github.com/actualbudget/actual/pull/3398) Prevent sync from saving to metadata.json unnecessarily — thanks @MikesGlitch
- [#3401](https://github.com/actualbudget/actual/pull/3401) Use react-aria-components' Button in filters. — thanks @joel-jeremy
- [#3406](https://github.com/actualbudget/actual/pull/3406) Adding retries into filesystem calls to mitigate locking issues caused by virus scanners. — thanks @MikesGlitch
- [#3408](https://github.com/actualbudget/actual/pull/3408) SyncedPrefs: refactor `ImportTransactions` usage of prefs to use a common hook. — thanks @MatissJanis
- [#3410](https://github.com/actualbudget/actual/pull/3410) SyncedPrefs: move `budgetType` back to metadata prefs. — thanks @MatissJanis
- [#3413](https://github.com/actualbudget/actual/pull/3413) Complete migration of all modals to react-aria-components Modal component. — thanks @joel-jeremy
- [#3423](https://github.com/actualbudget/actual/pull/3423) SyncedPrefs: move synced-preferences from metadata.json to the database. — thanks @MatissJanis
- [#3426](https://github.com/actualbudget/actual/pull/3426) Support translations in desktop-client/components/budget. — thanks @a-gradina
- [#3427](https://github.com/actualbudget/actual/pull/3427) SyncedPrefs: move budget type to synced preferences. — thanks @MatissJanis
- [#3430](https://github.com/actualbudget/actual/pull/3430) Support translations for component files — thanks @a-gradina
- [#3431](https://github.com/actualbudget/actual/pull/3431) Restart server silently when adding self signed cert and add some logs — thanks @MikesGlitch
- [#3443](https://github.com/actualbudget/actual/pull/3443) Reduce size of desktop packages — thanks @MikesGlitch
- [#3448](https://github.com/actualbudget/actual/pull/3448) Reduce budget table re-renders — thanks @joel-jeremy
- [#3450](https://github.com/actualbudget/actual/pull/3450) Cleanup react-aria packages and dedupe packages. — thanks @joel-jeremy
- [#3452](https://github.com/actualbudget/actual/pull/3452) Update synced account balance in db if available — thanks @matt-fidd
- [#3458](https://github.com/actualbudget/actual/pull/3458) SyncedPrefs: separate out MetadataPrefs and LocalPrefs in different storage locations. — thanks @MatissJanis
- [#3468](https://github.com/actualbudget/actual/pull/3468) Remove old updater logic — thanks @MikesGlitch
- [#3475](https://github.com/actualbudget/actual/pull/3475) Replacing the deprecated Electron file handler protocol with the new version — thanks @MikesGlitch
- [#3483](https://github.com/actualbudget/actual/pull/3483) File/variable renames: rollover budget -> envelope budget; report budget -> tracking budget. — thanks @MatissJanis
- [#3493](https://github.com/actualbudget/actual/pull/3493) Custom reports: added e2e tests to improve stability. — thanks @MatissJanis
- [#3497](https://github.com/actualbudget/actual/pull/3497) Fix running `yarn vrt:docker` on windows with git bash — thanks @UnderKoen
- [#3498](https://github.com/actualbudget/actual/pull/3498) e2e: improve rules test stability. — thanks @MatissJanis
- [#3503](https://github.com/actualbudget/actual/pull/3503) e2e: improve onboarding test stability. — thanks @MatissJanis
- [#3512](https://github.com/actualbudget/actual/pull/3512) e2e: improve rules test stability. — thanks @MatissJanis
- [#3513](https://github.com/actualbudget/actual/pull/3513) e2e: improve settings test stability. — thanks @MatissJanis
- [#3514](https://github.com/actualbudget/actual/pull/3514) Add category goal info to the budget prewarm list for faster loading of indicator colors. — thanks @youngcw
- [#3521](https://github.com/actualbudget/actual/pull/3521) Cleanup: Set theme-color at App level instead of per page — thanks @joel-jeremy
- [#3523](https://github.com/actualbudget/actual/pull/3523) e2e stability: wait for data-theme to switch before taking a screenshot — thanks @joel-jeremy
- [#3525](https://github.com/actualbudget/actual/pull/3525) Expose underlying exception source when a module fails to load. — thanks @qedi-r
- [#3526](https://github.com/actualbudget/actual/pull/3526) Fix electron build workflow for ubuntu-latest — thanks @matt-fidd
- [#3530](https://github.com/actualbudget/actual/pull/3530) [e2e] Fix the flaky "navigates back to start page by clicking on “no server” in an empty budget file test" from onboarding.test.js — thanks @joel-jeremy |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
⛔ Files ignored due to path filters (93)
upcoming-release-notes/3085.mdis excluded by!**/*.mdupcoming-release-notes/3119.mdis excluded by!**/*.mdupcoming-release-notes/3163.mdis excluded by!**/*.mdupcoming-release-notes/3277.mdis excluded by!**/*.mdupcoming-release-notes/3286.mdis excluded by!**/*.mdupcoming-release-notes/3288.mdis excluded by!**/*.mdupcoming-release-notes/3298.mdis excluded by!**/*.mdupcoming-release-notes/3311.mdis excluded by!**/*.mdupcoming-release-notes/3313.mdis excluded by!**/*.mdupcoming-release-notes/3325.mdis excluded by!**/*.mdupcoming-release-notes/3326.mdis excluded by!**/*.mdupcoming-release-notes/3332.mdis excluded by!**/*.mdupcoming-release-notes/3344.mdis excluded by!**/*.mdupcoming-release-notes/3352.mdis excluded by!**/*.mdupcoming-release-notes/3355.mdis excluded by!**/*.mdupcoming-release-notes/3356.mdis excluded by!**/*.mdupcoming-release-notes/3361.mdis excluded by!**/*.mdupcoming-release-notes/3363.mdis excluded by!**/*.mdupcoming-release-notes/3364.mdis excluded by!**/*.mdupcoming-release-notes/3367.mdis excluded by!**/*.mdupcoming-release-notes/3374.mdis excluded by!**/*.mdupcoming-release-notes/3377.mdis excluded by!**/*.mdupcoming-release-notes/3379.mdis excluded by!**/*.mdupcoming-release-notes/3380.mdis excluded by!**/*.mdupcoming-release-notes/3389.mdis excluded by!**/*.mdupcoming-release-notes/3391.mdis excluded by!**/*.mdupcoming-release-notes/3393.mdis excluded by!**/*.mdupcoming-release-notes/3394.mdis excluded by!**/*.mdupcoming-release-notes/3395.mdis excluded by!**/*.mdupcoming-release-notes/3396.mdis excluded by!**/*.mdupcoming-release-notes/3397.mdis excluded by!**/*.mdupcoming-release-notes/3398.mdis excluded by!**/*.mdupcoming-release-notes/3399.mdis excluded by!**/*.mdupcoming-release-notes/3400.mdis excluded by!**/*.mdupcoming-release-notes/3401.mdis excluded by!**/*.mdupcoming-release-notes/3406.mdis excluded by!**/*.mdupcoming-release-notes/3408.mdis excluded by!**/*.mdupcoming-release-notes/3410.mdis excluded by!**/*.mdupcoming-release-notes/3411.mdis excluded by!**/*.mdupcoming-release-notes/3412.mdis excluded by!**/*.mdupcoming-release-notes/3413.mdis excluded by!**/*.mdupcoming-release-notes/3417.mdis excluded by!**/*.mdupcoming-release-notes/3423.mdis excluded by!**/*.mdupcoming-release-notes/3426.mdis excluded by!**/*.mdupcoming-release-notes/3427.mdis excluded by!**/*.mdupcoming-release-notes/3430.mdis excluded by!**/*.mdupcoming-release-notes/3431.mdis excluded by!**/*.mdupcoming-release-notes/3432.mdis excluded by!**/*.mdupcoming-release-notes/3441.mdis excluded by!**/*.mdupcoming-release-notes/3443.mdis excluded by!**/*.mdupcoming-release-notes/3448.mdis excluded by!**/*.mdupcoming-release-notes/3450.mdis excluded by!**/*.mdupcoming-release-notes/3451.mdis excluded by!**/*.mdupcoming-release-notes/3452.mdis excluded by!**/*.mdupcoming-release-notes/3453.mdis excluded by!**/*.mdupcoming-release-notes/3456.mdis excluded by!**/*.mdupcoming-release-notes/3458.mdis excluded by!**/*.mdupcoming-release-notes/3459.mdis excluded by!**/*.mdupcoming-release-notes/3460.mdis excluded by!**/*.mdupcoming-release-notes/3462.mdis excluded by!**/*.mdupcoming-release-notes/3463.mdis excluded by!**/*.mdupcoming-release-notes/3468.mdis excluded by!**/*.mdupcoming-release-notes/3469.mdis excluded by!**/*.mdupcoming-release-notes/3472.mdis excluded by!**/*.mdupcoming-release-notes/3475.mdis excluded by!**/*.mdupcoming-release-notes/3478.mdis excluded by!**/*.mdupcoming-release-notes/3480.mdis excluded by!**/*.mdupcoming-release-notes/3483.mdis excluded by!**/*.mdupcoming-release-notes/3484.mdis excluded by!**/*.mdupcoming-release-notes/3487.mdis excluded by!**/*.mdupcoming-release-notes/3491.mdis excluded by!**/*.mdupcoming-release-notes/3492.mdis excluded by!**/*.mdupcoming-release-notes/3493.mdis excluded by!**/*.mdupcoming-release-notes/3495.mdis excluded by!**/*.mdupcoming-release-notes/3497.mdis excluded by!**/*.mdupcoming-release-notes/3498.mdis excluded by!**/*.mdupcoming-release-notes/3501.mdis excluded by!**/*.mdupcoming-release-notes/3503.mdis excluded by!**/*.mdupcoming-release-notes/3509.mdis excluded by!**/*.mdupcoming-release-notes/3510.mdis excluded by!**/*.mdupcoming-release-notes/3512.mdis excluded by!**/*.mdupcoming-release-notes/3513.mdis excluded by!**/*.mdupcoming-release-notes/3514.mdis excluded by!**/*.mdupcoming-release-notes/3515.mdis excluded by!**/*.mdupcoming-release-notes/3521.mdis excluded by!**/*.mdupcoming-release-notes/3523.mdis excluded by!**/*.mdupcoming-release-notes/3525.mdis excluded by!**/*.mdupcoming-release-notes/3526.mdis excluded by!**/*.mdupcoming-release-notes/3530.mdis excluded by!**/*.mdupcoming-release-notes/3535.mdis excluded by!**/*.mdupcoming-release-notes/3539.mdis excluded by!**/*.mdupcoming-release-notes/3540.mdis excluded by!**/*.mdupcoming-release-notes/3541.mdis excluded by!**/*.md
📒 Files selected for processing (3)
- packages/api/package.json (1 hunks)
- packages/desktop-client/package.json (1 hunks)
- packages/desktop-electron/package.json (1 hunks)
✅ Files skipped from review due to trivial changes (2)
- packages/desktop-client/package.json
- packages/desktop-electron/package.json
This was referenced Oct 2, 2024
MikesGlitch
approved these changes
Oct 2, 2024
joel-jeremy
approved these changes
Oct 3, 2024
matt-fidd
approved these changes
Oct 3, 2024
MatissJanis
added a commit
to actualbudget/docs
that referenced
this pull request
Oct 3, 2024
- web: actualbudget/actual#3545 - server: actualbudget/actual-server#472 - docs: #460 --------- Signed-off-by: check-spelling-bot <check-spelling-bot@users.noreply.github.com> Co-authored-by: Matiss Janis Aboltins <MatissJanis@users.noreply.github.com> Co-authored-by: Julian Dominguez-Schatz <julian.dominguezschatz@gmail.com>
This was referenced Oct 5, 2024
Closed
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.