Merge pull request #17 from Gauravbhatt19/fix/16

(#16) Resetting Alarms everytime Chrome starts
Gauravbhatt19 · Feb 9, 2021 · 8cc5626 · 8cc5626
2 parents 0fe8bd9 + 44955ff
commit 8cc5626
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 1 deletion.
diff --git a/background.js b/background.js
@@ -18,6 +18,10 @@ chrome.storage.sync.get(['checkedNotifications'], function(storedData) {
 			priority: 0
 		});
 	});
+
+	chrome.runtime.onStartup.addListener(function () {
+		setAlarms(storedCheckedNotifications);
+	});
 });
 
 function getObjectElement(haystack, key, value) {
@@ -28,3 +32,16 @@ function getObjectElement(haystack, key, value) {
 	}
 	return false;
 }
+
+function setAlarms(checkedNotifications){
+	chrome.alarms.clearAll();
+	checkedNotifications.forEach(function(notification){
+		chrome.alarms.create(
+			notification.id,
+			{
+				when: Date.now() + notification.cycleTime * 60000,
+				periodInMinutes: notification.cycleTime
+			}
+		);
+	});
+}
diff --git a/manifest.json b/manifest.json
@@ -1,7 +1,7 @@
 {
 	"manifest_version": 2,
 	"name": "Mind me please",
-	"version": "1.0",
+	"version": "1.0.2",
 	"description": "Health reminders while working on the desk",
 	"icons": {
 		"128": "assets/png/icon128.png",

diff --git a/popup/index.js b/popup/index.js
@@ -49,6 +49,7 @@ function getCheckedNotifications(storedCheckedNotifications) {
 					'id': checkedNotification.id,
 					'title': checkedNotification.title,
 					'quote': checkedNotification.quote,
+					'cycleTime': checkedNotification.cycleTime
 				});
 			}
 		}