[PR #5760] [MERGED] improve cleanup when opening multiple budgets through the API #6099

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/5760
Author: @matt-fidd
Created: 9/21/2025
Status: Merged
Merged: 9/21/2025
Merged by: @matt-fidd

Base: masterHead: start-services-error


📝 Commits (2)

📊 Changes

2 files changed (+8 additions, -1 deletions)

View changed files

📝 packages/loot-core/src/server/budgetfiles/app.ts (+2 -1)
upcoming-release-notes/5760.md (+6 -0)

📄 Description

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

I may have misunderstood something here, but loadBudget calls mainApp.startServices(), and closeBudget only called app.stopServices() so there were still some main services running when loadBudget tried to restart them for the new budget file.


🔄 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/5760 **Author:** [@matt-fidd](https://github.com/matt-fidd) **Created:** 9/21/2025 **Status:** ✅ Merged **Merged:** 9/21/2025 **Merged by:** [@matt-fidd](https://github.com/matt-fidd) **Base:** `master` ← **Head:** `start-services-error` --- ### 📝 Commits (2) - [`57a31f8`](https://github.com/actualbudget/actual/commit/57a31f82eb74fa96417093ebd431f216ad66bc4f) stop services before opening new budget file - [`6db4377`](https://github.com/actualbudget/actual/commit/6db43776512a0e3122cb5d27ff8b901d069f6b89) note ### 📊 Changes **2 files changed** (+8 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `packages/loot-core/src/server/budgetfiles/app.ts` (+2 -1) ➕ `upcoming-release-notes/5760.md` (+6 -0) </details> ### 📄 Description Fixes https://github.com/actualbudget/actual/issues/5441 I may have misunderstood something here, but loadBudget calls mainApp.startServices(), and closeBudget only called app.stopServices() so there were still some main services running when loadBudget tried to restart them for the new budget file. --- <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:23:53 -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#6099