Move default aggregator specification to separate module, to be expan…

…ded for #369
machawk1 · Jan 29, 2025 · 6f12b08 · 6f12b08
1 parent cb30e76
commit 6f12b08
Show file tree

Hide file tree

Showing 5 changed files with 25 additions and 9 deletions.
diff --git a/mink-plugin/MinkDefaults.js b/mink-plugin/MinkDefaults.js
@@ -0,0 +1,10 @@
+
+function MinkDefaults () {
+  this.defaultAggregators = [
+    'https://memgator.cs.odu.edu',
+    'https://aggregator.matkelly.com'
+  ]
+
+}
+
+export {MinkDefaults}
diff --git a/mink-plugin/content.js b/mink-plugin/content.js
@@ -430,6 +430,10 @@ chrome.runtime.onMessage.addListener(function (request, sender, sendResponse) {
       chrome.storage.local.set(storedHeaders, displayUIBasedOnContext)
     })
   }
+
+  if (request.method === 'setDefaultAggregators') {
+    console.log('TODO: set default aggregators from content.js')
+  }
 })
 
 function getMementos (uri) {

diff --git a/mink-plugin/mink.js b/mink-plugin/mink.js
@@ -10,6 +10,11 @@ const browserActionTitleNormal = 'Mink - Integrating the Live and Archived Web'
 const browserActionTitleNoMementos = 'Mink - No Mementos Available'
 const browserActionTitleIgnorelisted = 'Mink - Viewing Ignored Site'
 
+const defaultAggregators = [
+  'https://memgator.cs.odu.edu',
+  'https://aggregator.matkelly.com'
+]
+
 const badgeImagesDisabled = {
   38: chrome.runtime.getURL('images/minkLogo38_disabled.png'),
   19: chrome.runtime.getURL('images/minkLogo19_disabled.png')
@@ -781,10 +786,10 @@ chrome.action.onClicked.addListener(tab => {
 chrome.runtime.onInstalled.addListener(({ reason }) => {
   if (reason === 'install') {
     console.log('extension installed, reported via SW registration')
+    console.log('setting default aggregators')
+    chrome.storage.local.set({'aggregators': defaultAggregators})
     //chrome.storage.local.set({
     //  apiSuggestions: ['tabs', 'storage', 'scripting']
     //});
   }
-});
-
-
+})
diff --git a/mink-plugin/options.html b/mink-plugin/options.html
@@ -3,7 +3,7 @@
 <head>
     <title>Mink options</title>
     <script src="js/jquery-3.4.1.min.js"></script>
-    <script src="options.js" defer></script>
+    <script src="options.js" type="module" defer></script>
     <link rel="stylesheet" href="css/options.css">
 </head>
 <body>

diff --git a/mink-plugin/options.js b/mink-plugin/options.js
@@ -5,10 +5,7 @@ var debug = true
 const tmDropdownString = '<option>&nbsp;&nbsp;&darr; Mink has TimeMaps for... &darr;</option>'
 const tmDropdownNoTimemapsString = '<option>--- No TimeMaps available ---</option>'
 
-const defaultAggregators = [
-  'https://memgator.cs.odu.edu',
-  'https://aggregator.matkelly.com'
-]
+import {MinkDefaults} from './MinkDefaults.js'
 
 function restoreOptions () {
   chrome.storage.local.get('ignorelist').then(function (items) {
@@ -295,7 +292,7 @@ function clearAggregatorListInLocalStorage () {
 }
 
 function resetDefaultAggregators () {
-  chrome.storage.local.set({'aggregators': defaultAggregators})
+  chrome.storage.local.set({'aggregators': MinkDefaults.defaultAggregators})
 }
 
 function updateAggregatorsUIBasedOnLocalStorage () {