[PR #4375] [MERGED] 🐛 Fix Initializing the connection to the local database hang #31275

Closed
opened 2026-04-18 07:32:34 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/4375
Author: @MikesGlitch
Created: 2/13/2025
Status: Merged
Merged: 2/13/2025
Merged by: @MikesGlitch

Base: masterHead: fix/stuck-on-initializing-to-the-local-db


📝 Commits (2)

📊 Changes

3 files changed (+30 additions, -2 deletions)

View changed files

📝 packages/loot-core/src/platform/client/fetch/index.browser.ts (+5 -1)
📝 packages/loot-core/src/platform/server/connection/index.web.ts (+19 -1)
upcoming-release-notes/4375.md (+6 -0)

📄 Description

It was a race condition. The backend was loading in faster than the frontend. It was sending the connect message to the frontend and the frontend wasn't ready to receive it.

We can't guarantee the order that the frontend or backend will load in, so we have to add retries to deal with it.

Fixes this:

test

Fixes: https://github.com/actualbudget/actual/issues/2893


🔄 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/4375 **Author:** [@MikesGlitch](https://github.com/MikesGlitch) **Created:** 2/13/2025 **Status:** ✅ Merged **Merged:** 2/13/2025 **Merged by:** [@MikesGlitch](https://github.com/MikesGlitch) **Base:** `master` ← **Head:** `fix/stuck-on-initializing-to-the-local-db` --- ### 📝 Commits (2) - [`bb137a4`](https://github.com/actualbudget/actual/commit/bb137a45d75da01a7e7d59565f959dde13f7c2ca) fix initializing to the local db - [`27c3bdb`](https://github.com/actualbudget/actual/commit/27c3bdbbc93bf3807b3a6847f91b46548af00afe) release notes ### 📊 Changes **3 files changed** (+30 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `packages/loot-core/src/platform/client/fetch/index.browser.ts` (+5 -1) 📝 `packages/loot-core/src/platform/server/connection/index.web.ts` (+19 -1) ➕ `upcoming-release-notes/4375.md` (+6 -0) </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 --> It was a race condition. The backend was loading in faster than the frontend. It was sending the **_connect_** message to the frontend and the frontend wasn't ready to receive it. We can't guarantee the order that the frontend or backend will load in, so we have to add retries to deal with it. **Fixes this:** ![test](https://github.com/user-attachments/assets/71c5c50a-b9c4-4e7c-8f46-3da36499f1c1) Fixes: https://github.com/actualbudget/actual/issues/2893 --- <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-04-18 07:32:34 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#31275