Fix several issues

- visiting Translate.Next shows Pontoon Tools as not signed in, #97 - after installation errors are shown in console
MikkCZ · Aug 25, 2019 · f345874 · f345874
1 parent 7ba8bc8
commit f345874
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 9 deletions.
diff --git a/src/packages/background/ContextButtons.js b/src/packages/background/ContextButtons.js
@@ -26,10 +26,12 @@ class ContextButtons {
      * @private
      */
     _initMozillaWebsitesList(projects) {
-        this._mozillaWebsites = [];
-        Object.values(projects).forEach((project) =>
-            project.domains.forEach((domain) => this._mozillaWebsites.push(`https://${domain}`))
-        );
+        if (projects) {
+            this._mozillaWebsites = [];
+            Object.values(projects).forEach((project) =>
+                project.domains.forEach((domain) => this._mozillaWebsites.push(`https://${domain}`))
+            );
+        }
     }
 
     /**

diff --git a/src/packages/background/RemotePontoon.js b/src/packages/background/RemotePontoon.js
@@ -165,14 +165,10 @@ class RemotePontoon {
                 notificationsIdsFromPage,
             ]) => {
                 const notificationsInStorage = storageItem[dataKey];
-                if (!notificationsIdsFromPage.every(id => id in notificationsInStorage)) {
+                if (!notificationsInStorage || !notificationsIdsFromPage.every(id => id in notificationsInStorage)) {
                     this.updateNotificationsData();
                 }
             });
-        } else if (page.title === 'Translate.Next') {
-            // Translate.Next does not contain notifications in DOM
-        } else {
-            browser.storage.local.set({notificationsData: undefined});
         }
     }