[PR #4768] [MERGED] :electron: Remove ngrok from electron server embed #5574

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/4768
Author: @MikesGlitch
Created: 4/7/2025
Status: Merged
Merged: 4/7/2025
Merged by: @MikesGlitch

Base: masterHead: electron/remove-ngrok


📝 Commits (3)

📊 Changes

5 files changed (+8 additions, -186 deletions)

View changed files

📝 packages/desktop-electron/index.ts (+2 -41)
📝 packages/desktop-electron/package.json (+0 -1)
📝 packages/loot-core/src/types/prefs.d.ts (+0 -4)
upcoming-release-notes/4768.md (+6 -0)
📝 yarn.lock (+0 -140)

📄 Description

The plan is to move this to a plugin when plugins are available.

Removing it from here because it's a third party and it isn't sitting well with me that we're bundling it.

The user can still expose the server but they'll have to set it up manually until we have a plugin.


🔄 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/4768 **Author:** [@MikesGlitch](https://github.com/MikesGlitch) **Created:** 4/7/2025 **Status:** ✅ Merged **Merged:** 4/7/2025 **Merged by:** [@MikesGlitch](https://github.com/MikesGlitch) **Base:** `master` ← **Head:** `electron/remove-ngrok` --- ### 📝 Commits (3) - [`1f3cc26`](https://github.com/actualbudget/actual/commit/1f3cc26d22763ea4d3489b7331d59fdafa99ec5d) remove ngrok - [`fb48a25`](https://github.com/actualbudget/actual/commit/fb48a25c9a58c94034355e948c844bfff041aa75) lock - [`9f11372`](https://github.com/actualbudget/actual/commit/9f1137220dbc189b0d8e28a37ca10c19c657bef5) release notes ### 📊 Changes **5 files changed** (+8 additions, -186 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-electron/index.ts` (+2 -41) 📝 `packages/desktop-electron/package.json` (+0 -1) 📝 `packages/loot-core/src/types/prefs.d.ts` (+0 -4) ➕ `upcoming-release-notes/4768.md` (+6 -0) 📝 `yarn.lock` (+0 -140) </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. Try running yarn generate:release-notes *before* pushing your PR for an interactive experience. --> The plan is to move this to a plugin when plugins are available. Removing it from here because it's a third party and it isn't sitting well with me that we're bundling it. The user can still expose the server but they'll have to set it up manually until we have a plugin. --- <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:14:40 -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#5574