fix: setting default values and handle manifest error

#2
strumswell · Dec 7, 2022 · 6f0501a · 6f0501a
1 parent 0a06b76
commit 6f0501a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/Shared (Extension)/Resources/background.js b/Shared (Extension)/Resources/background.js
@@ -7,7 +7,13 @@ browser.runtime.onMessage.addListener((request, sender, sendResponse) => {
     // Retrieve action for settings change
     } else if (request.type === "GetStoreExecution"){
         browser.storage.local.get(request.id).then((res) => {
-            sendResponse({ checked: res[request.id] })
+            if(Object.keys(res).length > 0) {
+                sendResponse({ checked: res[request.id] })
+            } else {
+                // Set default value if not initialized yet
+                browser.storage.local.set({[request.id]: true})
+                sendResponse({ checked: true })
+            }
         })
     }
     return true;

diff --git a/Shared (Extension)/Resources/manifest.json b/Shared (Extension)/Resources/manifest.json
@@ -14,9 +14,8 @@
         "1024": "images/1024.png"
     },
     "background": {
-        "service_worker": "background.js"
+        "scripts": ["background.js"]
     },
-
     "content_scripts": [{
         "js": [ "content.js" ],
         "matches": [ "https://*.twitch.tv/*" ]