Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Performance regression: Accounts and Budget pages are slow #3881

Closed
2 tasks done
MikesGlitch opened this issue Nov 23, 2024 · 0 comments
Closed
2 tasks done

[Bug]: Performance regression: Accounts and Budget pages are slow #3881

MikesGlitch opened this issue Nov 23, 2024 · 0 comments
Labels
bug Something isn't working
Milestone

Comments

@MikesGlitch
Copy link
Contributor

MikesGlitch commented Nov 23, 2024

Verified issue does not already exist?

  • I have searched and found no existing issue
  • I will be providing steps how to reproduce the bug (in most cases this will also mean uploading a demo budget file)

What happened?

Navigating from an account transactions page to the budget page is extremely slow.

test

It was introduced by this PR: https://github.com/actualbudget/actual/pull/3864/files#diff-0305cff3699580e47a344c2d8bacf194dab7e51d925a125db9a83ad7a8b5bae1R69

When I change the deps back from:

  }, [sheetName, bindingObj.name, bindingObj.query]);

to:

  }, [sheetName, bindingObj.name]); 

I get the performance back, but it reintroduces the bug.

We need to find a way to bring the performance back without reintroducing the bug.

The performance is more obvious on the Electron app but also affects the web app.

Where are you hosting Actual?

Desktop App (Electron)

What browsers are you seeing the problem on?

Chrome, Microsoft Edge, Desktop App (Electron)

Operating System

Windows 10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant