Remove squirrel dependency, error in default telemetry setting

pierr3 · May 4, 2024 · 16b5c67 · 16b5c67
1 parent 9a75373
commit 16b5c67
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 29 deletions.
diff --git a/forge.config.ts b/forge.config.ts
@@ -29,6 +29,7 @@ const config: ForgeConfig = {
       config: {
         installerIcon: "resources/AppIcon/AppIcon.ico",
         uninstallerIcon: "resources/AppIcon/AppIcon.ico",
+        icon: "resources/AppIcon/AppIcon.ico",
         license: "LICENSE",
       },
     },

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -65,7 +65,6 @@
     "bootstrap": "^5.3.3",
     "bootstrap-scss": "^5.3.3",
     "clsx": "^2.1.0",
-    "electron-squirrel-startup": "^1.0.0",
     "electron-store": "^8.2.0",
     "i": "^0.3.7",
     "npm": "^10.5.1",
@@ -84,4 +83,4 @@
   "engines": {
     "node": ">=20.0.0 <=21.0.0"
   }
-}
+}
diff --git a/src/index.ts b/src/index.ts
@@ -25,9 +25,6 @@ Sentry.init({
   sendDefaultPii: false,
 });
 
-// Handle creating/removing shortcuts on Windows when installing/uninstalling.
-if (require("electron-squirrel-startup")) app.quit();
-
 let version = "";
 let mainWindow: BrowserWindow;
 
@@ -169,7 +166,7 @@ app.on("ready", () => {
     const sclient = Sentry.getClient();
     if (sclient) {
       // Disable sentry in debug always
-      sclient.getOptions().enabled = app.isPackaged
+      sclient.getOptions().enabled = !app.isPackaged
         ? false
         : currentConfiguration.consentedToTelemetry;
     } else {
@@ -384,7 +381,7 @@ ipcMain.handle("change-telemetry", (_, enabled: boolean) => {
 });
 
 ipcMain.handle("should-enable-renderer-telemetry", () => {
-  return app.isPackaged ? false : currentConfiguration.consentedToTelemetry;
+  return !app.isPackaged ? false : currentConfiguration.consentedToTelemetry;
 });
 
 ipcMain.handle(