Merge branch 'main' into fix-firefox-world

freelawproject · Oct 15, 2024 · f743455 · f743455
2 parents 8dc807a + 95084d5
commit f743455
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -15,6 +15,7 @@ Fixes:
  - Refines the generateFileName method to accurately compute zip file names ([366](https://github.com/freelawproject/recap/issues/366), [399](https://github.com/freelawproject/recap-chrome/pull/399)).
  - Improves the reliability of PACER case ID retrieval on attachment pages ([369](https://github.com/freelawproject/recap/issues/369), [400](https://github.com/freelawproject/recap-chrome/pull/400)).
  - Fix setDefaultOptions in updateToolbarButton([403](https://github.com/freelawproject/recap-chrome/pull/403))
+ - Ensure we call sendResponse even when notifications are disabled([405](https://github.com/freelawproject/recap-chrome/pull/405))
  - Use chrome.scripting.ExecutionWorld.MAIN for firefox compatibility ([404](https://github.com/freelawproject/recap-chrome/pull/404))
 
 

diff --git a/src/utils/background_notifier.js b/src/utils/background_notifier.js
@@ -64,8 +64,8 @@ export const handleBackgroundNotification = (req, sender, sendResponse) => {
       break;
     case 'showUpload':
       chrome.storage.local.get('options', (items) => {
-        if (!items || !items.options.show_notifications) return;
-        showNotification(title, message);
+        if (items && items.options.show_notifications)
+          showNotification(title, message);
         sendResponse({ status: 'success ' });
       });
       break;