Cleanup comments and variables

Author: vivek-nexus

extension/background.js

@@ -10,10 +10,10 @@ const timeFormat = {
 // Listen for extension updates
 chrome.runtime.onUpdateAvailable.addListener((details) => {
     console.log('Extension update available:', details.version)
-    // Check if there's an active meeting
-    chrome.storage.local.get(["meetingTabId"], function (data) {
-        if (data.meetingTabId) {
-            // There's an active meeting, defer the update
+    // Check if there is an active meeting
+    chrome.storage.local.get(["meetingTabId"], function (result) {
+        if (result.meetingTabId) {
+            // There is an active meeting, defer the update
             console.log('Caught event, which will defer this update attempt')
         } else {
             // No active meeting, apply the update immediately
@@ -62,7 +62,7 @@ chrome.runtime.onMessage.addListener(function (message, sender, sendResponse) {
             })
     }
 
-    if (message.type == "recover_last_transcript_and_download") {
+    if (message.type == "recover_last_transcript") {
         downloadAndPostWebhook()
         sendResponse({ message: "Recovery process started" })
     }

extension/content.js

@@ -586,20 +586,20 @@ function recoverLastMeeting() {
 
           // Last meeting was not processed for some reason. Need to recover that data, process and download it.
           if (result.meetingStartTimestamp !== meetingToDownload.meetingStartTimestamp) {
-            // Silent failure if last meeting was an empty meeting
+            // Silent failure if last meeting is an empty meeting
             chrome.runtime.sendMessage({
-              type: "recover_last_transcript_and_download",
+              type: "recover_last_transcript",
             }, function (response) {
               console.log(response)
               resolve()
               return
             })
           }
         }
-        // First meeting itself ended in a disaster. Need to recover that data, process and download it. Also handles recoveries of versions where "meetingStartTimeStamp" was used, because result.meetings will always be undefined in those versions.
+        // First meeting itself ended in a disaster. Need to recover that data, process and download it. Also handle recoveries of versions where "meetingStartTimeStamp" was used, because result.meetings will always be undefined in those versions.
         else {
           chrome.runtime.sendMessage({
-            type: "recover_last_transcript_and_download",
+            type: "recover_last_transcript",
           }, function (response) {
             console.log(response)
             resolve()

extension/popup.js

@@ -23,14 +23,15 @@ window.onload = function () {
   lastMeetingTranscriptLink.addEventListener("click", () => {
     chrome.storage.local.get(["meetings", "meetingStartTimestamp", "meetingStartTimeStamp"], function (result) {
       // Check if user ever attended a meeting
+      // Backward compatible chrome storage variable. Old name "meetingStartTimeStamp". 
       if (result.meetingStartTimestamp || result.meetingStartTimeStamp) {
         if (result.meetings && (result.meetings.length > 0)) {
 
           const meetingToDownload = result.meetings[result.meetings.length - 1]
 
           // Check if last meeting was successfully processed and added to meetings
           if (result.meetingStartTimestamp === meetingToDownload.meetingStartTimestamp) {
-            // Silent failure if last meeting was an empty meeting
+            // Silent failure if last meeting is an empty meeting
             chrome.runtime.sendMessage({
               type: "download_transcript_at_index",
               index: result.meetings.length - 1
@@ -40,18 +41,18 @@ window.onload = function () {
           }
           // Last meeting was not processed for some reason. Need to recover that data, process and download it.
           else {
-            // Silent failure if last meeting was an empty meeting
+            // Silent failure if last meeting is an empty meeting
             chrome.runtime.sendMessage({
-              type: "recover_last_transcript_and_download",
+              type: "recover_last_transcript",
             }, function (response) {
               console.log(response)
             })
           }
         }
-        // First meeting itself ended in a disaster. Need to recover that data, process and download it. Also handles recoveries of versions where "meetingStartTimeStamp" was used, because result.meetings will always be undefined in those versions.
+        // First meeting itself ended in a disaster. Need to recover that data, process and download it. Also handle recoveries of versions where "meetingStartTimeStamp" was used, because result.meetings will always be undefined in those versions.
         else {
           chrome.runtime.sendMessage({
-            type: "recover_last_transcript_and_download",
+            type: "recover_last_transcript",
           }, function (response) {
             console.log(response)
           })