[PR #6978] [WIP] Redirect the user to the budget page regardless of accounts #6859

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

Original Pull Request: https://github.com/actualbudget/actual/pull/6978

State: closed
Merged: No


Fixes #6977

The issue is that we are checking explicitly for this condition. So, if users don't have accounts setup, which is true when they are creating a fresh budget, we redirect them to "all accounts" to prompt them to create new accounts. Based on the issue, I think we can remove this check to redirect them to the budget page.

https://github.com/actualbudget/actual/blob/master/packages/desktop-client/src/components/FinancesApp.tsx#L248-L260


Bundle Stats

Bundle Files count Total bundle size % Changed
desktop-client 28 14.78 MB → 14.8 MB (+19.71 kB) +0.13%
loot-core 1 5.86 MB 0%
api 1 4.4 MB 0%
View detailed bundle stats

desktop-client

Total

Files count Total bundle size % Changed
28 14.78 MB → 14.8 MB (+19.71 kB) +0.13%
Changeset
File Δ Size
locale/ca.json 📈 +18.21 kB (+18.79%) 96.92 kB → 115.14 kB
locale/en.json 📈 +1.67 kB (+1.01%) 165.58 kB → 167.25 kB
src/components/FinancesApp.tsx 📉 -172 B (-0.90%) 18.59 kB → 18.42 kB
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger

Asset File Size % Changed
static/js/ca.js 96.92 kB → 115.14 kB (+18.21 kB) +18.79%
static/js/en.js 165.58 kB → 167.25 kB (+1.67 kB) +1.01%

Smaller

Asset File Size % Changed
static/js/index.js 9.52 MB → 9.51 MB (-172 B) -0.00%

Unchanged

Asset File Size % Changed
static/js/indexeddb-main-thread-worker-e59fee74.js 12.94 kB 0%
static/js/workbox-window.prod.es5.js 5.64 kB 0%
static/js/da.js 106.62 kB 0%
static/js/de.js 180.44 kB 0%
static/js/en-GB.js 7.18 kB 0%
static/js/es.js 173.83 kB 0%
static/js/fr.js 179.97 kB 0%
static/js/it.js 171.44 kB 0%
static/js/nb-NO.js 157.23 kB 0%
static/js/nl.js 106.65 kB 0%
static/js/pl.js 88.64 kB 0%
static/js/pt-BR.js 154.57 kB 0%
static/js/sv.js 78.2 kB 0%
static/js/th.js 182.35 kB 0%
static/js/uk.js 215.11 kB 0%
static/js/resize-observer.js 18.37 kB 0%
static/js/BackgroundImage.js 120.54 kB 0%
static/js/ReportRouter.js 1.13 MB 0%
static/js/narrow.js 638.75 kB 0%
static/js/TransactionList.js 106.13 kB 0%
static/js/wide.js 165.25 kB 0%
static/js/AppliedFilters.js 9.71 kB 0%
static/js/usePayeeRuleCounts.js 10.05 kB 0%
static/js/useTransactionBatchActions.js 13.23 kB 0%
static/js/FormulaEditor.js 1.04 MB 0%

loot-core

Total

Files count Total bundle size % Changed
1 5.86 MB 0%
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger
No assets were bigger

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
kcab.worker.92NpqIpU.js 5.86 MB 0%

api

Total

Files count Total bundle size % Changed
1 4.4 MB 0%
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger
No assets were bigger

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
bundle.api.js 4.4 MB 0%
**Original Pull Request:** https://github.com/actualbudget/actual/pull/6978 **State:** closed **Merged:** No --- Fixes #6977 The issue is that we are checking explicitly for this condition. So, if users don't have accounts setup, which is true when they are creating a fresh budget, we redirect them to "all accounts" to prompt them to create new accounts. Based on the issue, I think we can remove this check to redirect them to the budget page. https://github.com/actualbudget/actual/blob/master/packages/desktop-client/src/components/FinancesApp.tsx#L248-L260 <!--- actual-bot-sections ---> <hr /> <!--- bundlestats-action-comment key:combined start ---> ### Bundle Stats Bundle | Files count | Total bundle size | % Changed ------ | ----------- | ----------------- | --------- desktop-client | 28 | 14.78 MB → 14.8 MB (+19.71 kB) | +0.13% loot-core | 1 | 5.86 MB | 0% api | 1 | 4.4 MB | 0% <details> <summary>View detailed bundle stats</summary> #### desktop-client **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 28 | 14.78 MB → 14.8 MB (+19.71 kB) | +0.13% <details> <summary>Changeset</summary> File | Δ | Size ---- | - | ---- `locale/ca.json` | 📈 +18.21 kB (+18.79%) | 96.92 kB → 115.14 kB `locale/en.json` | 📈 +1.67 kB (+1.01%) | 165.58 kB → 167.25 kB `src/components/FinancesApp.tsx` | 📉 -172 B (-0.90%) | 18.59 kB → 18.42 kB </details> <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** Asset | File Size | % Changed ----- | --------- | --------- static/js/ca.js | 96.92 kB → 115.14 kB (+18.21 kB) | +18.79% static/js/en.js | 165.58 kB → 167.25 kB (+1.67 kB) | +1.01% **Smaller** Asset | File Size | % Changed ----- | --------- | --------- static/js/index.js | 9.52 MB → 9.51 MB (-172 B) | -0.00% **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- static/js/indexeddb-main-thread-worker-e59fee74.js | 12.94 kB | 0% static/js/workbox-window.prod.es5.js | 5.64 kB | 0% static/js/da.js | 106.62 kB | 0% static/js/de.js | 180.44 kB | 0% static/js/en-GB.js | 7.18 kB | 0% static/js/es.js | 173.83 kB | 0% static/js/fr.js | 179.97 kB | 0% static/js/it.js | 171.44 kB | 0% static/js/nb-NO.js | 157.23 kB | 0% static/js/nl.js | 106.65 kB | 0% static/js/pl.js | 88.64 kB | 0% static/js/pt-BR.js | 154.57 kB | 0% static/js/sv.js | 78.2 kB | 0% static/js/th.js | 182.35 kB | 0% static/js/uk.js | 215.11 kB | 0% static/js/resize-observer.js | 18.37 kB | 0% static/js/BackgroundImage.js | 120.54 kB | 0% static/js/ReportRouter.js | 1.13 MB | 0% static/js/narrow.js | 638.75 kB | 0% static/js/TransactionList.js | 106.13 kB | 0% static/js/wide.js | 165.25 kB | 0% static/js/AppliedFilters.js | 9.71 kB | 0% static/js/usePayeeRuleCounts.js | 10.05 kB | 0% static/js/useTransactionBatchActions.js | 13.23 kB | 0% static/js/FormulaEditor.js | 1.04 MB | 0% </div> </details> --- #### loot-core **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 1 | 5.86 MB | 0% <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- kcab.worker.92NpqIpU.js | 5.86 MB | 0% </div> </details> --- #### api **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 1 | 4.4 MB | 0% <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- bundle.api.js | 4.4 MB | 0% </div> </details> </details> <!--- bundlestats-action-comment key:combined end --->
GiteaMirror added the pull-request label 2026-02-28 21:33:41 -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#6859