Merge pull request #220 from NIAEFEUP/fix/notifications-load

Fix Notifications Load
NIAEFEUP · Nov 29, 2024 · d45dffc · d45dffc
2 parents 67a19f6 + c80f71d
commit d45dffc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/background.js b/background.js
@@ -1,3 +1,4 @@
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 const sigarraRegex = /.*:\/\/sigarra\.up\.pt\/feup\/.*/;
 
 // Add default values for each option here
@@ -63,6 +64,6 @@ chrome.runtime.onMessage.addListener(async (message, sender, sendResponse) => {
     }
 });
 
-chrome.permissions.onRemoved.addListener((permissions) => {
+chrome.permissions.onRemoved.addListener(() => {
     //TODO:
 });
diff --git a/content-scripts/components/Navbar/HeaderAuthentication.tsx b/content-scripts/components/Navbar/HeaderAuthentication.tsx
@@ -18,6 +18,7 @@ const Authentication = ({ auth }: Props) => {
                 <Button
                     icon="ri-notification-line"
                     radius="full"
+                    id="se-auth-notifications-button"
                     onclick={() => togglePopover("se-auth-notifications-menu")}
                     className={auth.hasNotifications ? "se-badge" : ""}
                 />