Fix #6 - optional notifications

Fixes a bug in the mostly-complete implementation from https://github.com/jgmize/export-tabs-urls/tree/optional-notifications Also address: * alct#16 * alct#35
clach04 · Dec 3, 2023 · 736f9fa · 736f9fa
1 parent fa633c0
commit 736f9fa
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/options/options.js b/options/options.js
@@ -89,7 +89,7 @@ function saveOptions () {
       ignorePinned: optionsIgnorePinned.checked,
       formatCustom: optionsFormatCustom.value,
       filterTabs: optionsFilterTabs.checked,
-      notifications: optionsNotifications.checked,
+      enableNotifications: optionsNotifications.checked,
       customHeader: optionsCustomHeader.value
     }
   })

diff --git a/popup/popup.js b/popup/popup.js
@@ -241,6 +241,6 @@ function getOptions () {
     optionsFormatCustom = items.options.formatCustom
     optionsFilterTabs = items.options.filterTabs
     optionsCustomHeader = items.options.customHeader
-    optionsNotifications = items.options.notifications
+    optionsNotifications = items.options.enableNotifications
   })
 }
diff --git a/shared/shared.js b/shared/shared.js
@@ -3,6 +3,7 @@ var w = window
 
 var defaultOptions = {
   'options': {
+    enableNotifications: true,
     ignoreNonHTTP: true,
     ignorePinned: false,
     formatCustom: '',