From 5c01b5df5c4a36097682220b53ad686966dd5077 Mon Sep 17 00:00:00 2001
From: Afonso Jorge Ramos <afonsojorgeramos@gmail.com>
Date: Tue, 30 Dec 2025 04:21:55 +0100
Subject: [PATCH 1/3] feat: add 'Show read notifications' setting (#708)

---
 src/renderer/__mocks__/state-mocks.ts         |  1 +
 .../settings/NotificationSettings.test.tsx    | 16 +++++
 .../settings/NotificationSettings.tsx         | 21 +++++++
 src/renderer/context/defaults.ts              |  1 +
 .../__snapshots__/Settings.test.tsx.snap      | 61 +++++++++++++++++--
 src/renderer/types.ts                         |  1 +
 .../notifications/filters/filter.test.ts      | 28 +++++++++
 .../utils/notifications/filters/filter.ts     |  5 ++
 8 files changed, 128 insertions(+), 6 deletions(-)

diff --git a/src/renderer/__mocks__/state-mocks.ts b/src/renderer/__mocks__/state-mocks.ts
index 63ad1f1c0..9d05eff33 100644
--- a/src/renderer/__mocks__/state-mocks.ts
+++ b/src/renderer/__mocks__/state-mocks.ts
@@ -43,6 +43,7 @@ const mockNotificationSettings: NotificationSettingsState = {
   showPills: true,
   showNumber: true,
   participating: false,
+  showReadNotifications: true,
   markAsDoneOnOpen: false,
   markAsDoneOnUnsubscribe: false,
   delayNotificationState: false,
diff --git a/src/renderer/components/settings/NotificationSettings.test.tsx b/src/renderer/components/settings/NotificationSettings.test.tsx
index 648bf8c76..3fc3650a3 100644
--- a/src/renderer/components/settings/NotificationSettings.test.tsx
+++ b/src/renderer/components/settings/NotificationSettings.test.tsx
@@ -271,6 +271,22 @@ describe('renderer/components/settings/NotificationSettings.tsx', () => {
     );
   });
 
+  it('should toggle the showReadNotifications checkbox', async () => {
+    await act(async () => {
+      renderWithAppContext(<NotificationSettings />, {
+        updateSetting: updateSettingMock,
+      });
+    });
+
+    await userEvent.click(screen.getByTestId('checkbox-showReadNotifications'));
+
+    expect(updateSettingMock).toHaveBeenCalledTimes(1);
+    expect(updateSettingMock).toHaveBeenCalledWith(
+      'showReadNotifications',
+      false,
+    );
+  });
+
   it('should toggle the markAsDoneOnOpen checkbox', async () => {
     await act(async () => {
       renderWithAppContext(<NotificationSettings />, {
diff --git a/src/renderer/components/settings/NotificationSettings.tsx b/src/renderer/components/settings/NotificationSettings.tsx
index 66bb02de7..7e1c170c1 100644
--- a/src/renderer/components/settings/NotificationSettings.tsx
+++ b/src/renderer/components/settings/NotificationSettings.tsx
@@ -328,6 +328,27 @@ export const NotificationSettings: FC = () => {
           }
         />
 
+        <Checkbox
+          checked={settings.showReadNotifications}
+          label="Show read notifications"
+          name="showReadNotifications"
+          onChange={(evt) =>
+            updateSetting('showReadNotifications', evt.target.checked)
+          }
+          tooltip={
+            <Stack direction="vertical" gap="condensed">
+              <Text>
+                When <Text as="u">checked</Text>, {APPLICATION.NAME} will
+                display both read and unread notifications.
+              </Text>
+              <Text>
+                When <Text as="u">unchecked</Text>, {APPLICATION.NAME} will only
+                display unread notifications.
+              </Text>
+            </Stack>
+          }
+        />
+
         <Checkbox
           checked={settings.markAsDoneOnOpen}
           label="Mark as done on open"
diff --git a/src/renderer/context/defaults.ts b/src/renderer/context/defaults.ts
index b37fc260a..5ab2ae413 100644
--- a/src/renderer/context/defaults.ts
+++ b/src/renderer/context/defaults.ts
@@ -36,6 +36,7 @@ const defaultNotificationSettings: NotificationSettingsState = {
   showPills: true,
   showNumber: true,
   participating: false,
+  showReadNotifications: true,
   markAsDoneOnOpen: false,
   markAsDoneOnUnsubscribe: false,
   delayNotificationState: false,
diff --git a/src/renderer/routes/__snapshots__/Settings.test.tsx.snap b/src/renderer/routes/__snapshots__/Settings.test.tsx.snap
index a49af8031..3fba1e5aa 100644
--- a/src/renderer/routes/__snapshots__/Settings.test.tsx.snap
+++ b/src/renderer/routes/__snapshots__/Settings.test.tsx.snap
@@ -1098,6 +1098,55 @@ exports[`renderer/routes/Settings.tsx should render itself & its children 1`] =
               </svg>
             </button>
           </div>
+          <div
+            class="text-sm prc-Stack-Stack-UQ9k6"
+            data-align="center"
+            data-direction="horizontal"
+            data-gap="condensed"
+            data-justify="start"
+            data-padding="none"
+            data-wrap="nowrap"
+          >
+            <input
+              checked=""
+              class="size-4 rounded-sm cursor-pointer"
+              data-testid="checkbox-showReadNotifications"
+              id="showReadNotifications"
+              type="checkbox"
+            />
+            <label
+              class="font-medium text-gitify-font cursor-pointer"
+              for="showReadNotifications"
+            >
+              Show read notifications
+            </label>
+            <button
+              aria-expanded="false"
+              aria-haspopup="true"
+              aria-label="tooltip-showReadNotifications"
+              data-testid="tooltip-tooltip-showReadNotifications"
+              id="tooltip-showReadNotifications"
+              tabindex="0"
+              type="button"
+            >
+              <svg
+                aria-hidden="true"
+                class="octicon octicon-question text-gitify-tooltip-icon"
+                display="inline-block"
+                fill="currentColor"
+                focusable="false"
+                height="16"
+                overflow="visible"
+                style="vertical-align: text-bottom;"
+                viewBox="0 0 16 16"
+                width="16"
+              >
+                <path
+                  d="M0 8a8 8 0 1 1 16 0A8 8 0 0 1 0 8Zm8-6.5a6.5 6.5 0 1 0 0 13 6.5 6.5 0 0 0 0-13ZM6.92 6.085h.001a.749.749 0 1 1-1.342-.67c.169-.339.436-.701.849-.977C6.845 4.16 7.369 4 8 4a2.756 2.756 0 0 1 1.637.525c.503.377.863.965.863 1.725 0 .448-.115.83-.329 1.15-.205.307-.47.513-.692.662-.109.072-.22.138-.313.195l-.006.004a6.24 6.24 0 0 0-.26.16.952.952 0 0 0-.276.245.75.75 0 0 1-1.248-.832c.184-.264.42-.489.692-.661.103-.067.207-.132.313-.195l.007-.004c.1-.061.182-.11.258-.161a.969.969 0 0 0 .277-.245C8.96 6.514 9 6.427 9 6.25a.612.612 0 0 0-.262-.525A1.27 1.27 0 0 0 8 5.5c-.369 0-.595.09-.74.187a1.01 1.01 0 0 0-.34.398ZM9 11a1 1 0 1 1-2 0 1 1 0 0 1 2 0Z"
+                />
+              </svg>
+            </button>
+          </div>
           <div
             class="text-sm prc-Stack-Stack-UQ9k6"
             data-align="center"
@@ -1943,7 +1992,7 @@ exports[`renderer/routes/Settings.tsx should render itself & its children 1`] =
         data-wrap="nowrap"
       >
         <button
-          aria-describedby="_r_12_"
+          aria-describedby="_r_13_"
           class="prc-Button-ButtonBase-9n-Xk"
           data-loading="false"
           data-no-visuals="true"
@@ -1971,7 +2020,7 @@ exports[`renderer/routes/Settings.tsx should render itself & its children 1`] =
           aria-hidden="true"
           class="prc-TooltipV2-Tooltip-tLeuB"
           data-direction="ne"
-          id="_r_12_"
+          id="_r_13_"
           popover="auto"
           role="tooltip"
         >
@@ -1988,7 +2037,7 @@ exports[`renderer/routes/Settings.tsx should render itself & its children 1`] =
         data-wrap="nowrap"
       >
         <button
-          aria-describedby="_r_14_"
+          aria-describedby="_r_15_"
           aria-label="Accounts"
           class="prc-Button-ButtonBase-9n-Xk prc-Button-IconButton-fyge7"
           data-component="IconButton"
@@ -2020,14 +2069,14 @@ exports[`renderer/routes/Settings.tsx should render itself & its children 1`] =
           aria-hidden="true"
           class="prc-TooltipV2-Tooltip-tLeuB"
           data-direction="n"
-          id="_r_14_"
+          id="_r_15_"
           popover="auto"
           role="tooltip"
         >
           Accounts
         </span>
         <button
-          aria-describedby="_r_16_"
+          aria-describedby="_r_17_"
           aria-label="Quit Gitify"
           class="prc-Button-ButtonBase-9n-Xk prc-Button-IconButton-fyge7"
           data-component="IconButton"
@@ -2059,7 +2108,7 @@ exports[`renderer/routes/Settings.tsx should render itself & its children 1`] =
           aria-hidden="true"
           class="prc-TooltipV2-Tooltip-tLeuB"
           data-direction="nw"
-          id="_r_16_"
+          id="_r_17_"
           popover="auto"
           role="tooltip"
         >
diff --git a/src/renderer/types.ts b/src/renderer/types.ts
index 7939504a5..cff635f35 100644
--- a/src/renderer/types.ts
+++ b/src/renderer/types.ts
@@ -109,6 +109,7 @@ export interface NotificationSettingsState {
   showPills: boolean;
   showNumber: boolean;
   participating: boolean;
+  showReadNotifications: boolean;
   markAsDoneOnOpen: boolean;
   markAsDoneOnUnsubscribe: boolean;
   delayNotificationState: boolean;
diff --git a/src/renderer/utils/notifications/filters/filter.test.ts b/src/renderer/utils/notifications/filters/filter.test.ts
index 3d53bcef6..139860547 100644
--- a/src/renderer/utils/notifications/filters/filter.test.ts
+++ b/src/renderer/utils/notifications/filters/filter.test.ts
@@ -63,6 +63,34 @@ describe('renderer/utils/notifications/filters/filter.ts', () => {
     ];
 
     describe('filterBaseNotifications', () => {
+      it('should show all notifications when showReadNotifications is enabled', () => {
+        const notifications = [
+          { ...mockNotifications[0], unread: true },
+          { ...mockNotifications[1], unread: false },
+        ];
+        const result = filterBaseNotifications(notifications, {
+          ...mockSettings,
+          showReadNotifications: true,
+        });
+
+        expect(result.length).toBe(2);
+        expect(result).toEqual(notifications);
+      });
+
+      it('should filter out read notifications when showReadNotifications is disabled', () => {
+        const notifications = [
+          { ...mockNotifications[0], unread: true },
+          { ...mockNotifications[1], unread: false },
+        ];
+        const result = filterBaseNotifications(notifications, {
+          ...mockSettings,
+          showReadNotifications: false,
+        });
+
+        expect(result.length).toBe(1);
+        expect(result[0].unread).toBe(true);
+      });
+
       it('should filter notifications by subject type when provided', async () => {
         mockNotifications[0].subject.type = 'Issue';
         mockNotifications[1].subject.type = 'PullRequest';
diff --git a/src/renderer/utils/notifications/filters/filter.ts b/src/renderer/utils/notifications/filters/filter.ts
index a48619093..bc8b713b0 100644
--- a/src/renderer/utils/notifications/filters/filter.ts
+++ b/src/renderer/utils/notifications/filters/filter.ts
@@ -24,6 +24,11 @@ export function filterBaseNotifications(
   return notifications.filter((notification) => {
     let passesFilters = true;
 
+    // Filter out read notifications if showReadNotifications is disabled
+    if (!settings.showReadNotifications && !notification.unread) {
+      return false;
+    }
+
     // Apply base qualifier include/exclude filters (org, repo, etc.)
     for (const qualifier of BASE_SEARCH_QUALIFIERS) {
       if (!passesFilters) {

From a94c5246dd0f7e08e34b39bb73a952c551704348 Mon Sep 17 00:00:00 2001
From: Afonso Jorge Ramos <afonsojorgeramos@gmail.com>
Date: Tue, 30 Dec 2025 14:34:30 +0100
Subject: [PATCH 2/3] fix: hide mark as read button on read notifications

---
 .../notifications/NotificationRow.test.tsx     | 18 ++++++++++++++++++
 .../notifications/NotificationRow.tsx          |  1 +
 2 files changed, 19 insertions(+)

diff --git a/src/renderer/components/notifications/NotificationRow.test.tsx b/src/renderer/components/notifications/NotificationRow.test.tsx
index 14ed55fc6..58e22aadc 100644
--- a/src/renderer/components/notifications/NotificationRow.test.tsx
+++ b/src/renderer/components/notifications/NotificationRow.test.tsx
@@ -154,6 +154,24 @@ describe('renderer/components/notifications/NotificationRow.tsx', () => {
       expect(markNotificationsAsReadMock).toHaveBeenCalledTimes(1);
     });
 
+    it('should hide mark as read button when notification is already read', async () => {
+      const readNotification = {
+        ...mockSingleNotification,
+        unread: false,
+      };
+
+      const props = {
+        notification: readNotification,
+        account: mockGitHubCloudAccount,
+      };
+
+      renderWithAppContext(<NotificationRow {...props} />);
+
+      expect(
+        screen.queryByTestId('notification-mark-as-read'),
+      ).not.toBeInTheDocument();
+    });
+
     it('should mark notifications as done', async () => {
       const markNotificationsAsDoneMock = jest.fn();
 
diff --git a/src/renderer/components/notifications/NotificationRow.tsx b/src/renderer/components/notifications/NotificationRow.tsx
index c158689c6..6f61ea8ba 100644
--- a/src/renderer/components/notifications/NotificationRow.tsx
+++ b/src/renderer/components/notifications/NotificationRow.tsx
@@ -148,6 +148,7 @@ export const NotificationRow: FC<NotificationRowProps> = ({
 
           <HoverButton
             action={actionMarkAsRead}
+            enabled={!isNotificationRead}
             icon={ReadIcon}
             label="Mark as read"
             testid="notification-mark-as-read"

From e7c223a4e72632d3636f83f7b175f59d13c6d69c Mon Sep 17 00:00:00 2001
From: Afonso Jorge Ramos <afonsojorgeramos@gmail.com>
Date: Tue, 30 Dec 2025 16:52:49 +0100
Subject: [PATCH 3/3] fix: fetch read notifications from API when setting
 enabled

---
 src/renderer/__mocks__/state-mocks.ts         |  2 +-
 .../NotificationRow.test.tsx.snap             | 58 -------------------
 .../settings/NotificationSettings.test.tsx    |  2 +-
 .../settings/NotificationSettings.tsx         | 10 +++-
 src/renderer/context/defaults.ts              |  2 +-
 src/renderer/utils/api/client.test.ts         | 38 ++++++++++++
 src/renderer/utils/api/client.ts              |  4 ++
 .../notifications/filters/filter.test.ts      |  6 ++
 8 files changed, 58 insertions(+), 64 deletions(-)

diff --git a/src/renderer/__mocks__/state-mocks.ts b/src/renderer/__mocks__/state-mocks.ts
index 9d05eff33..15f057f96 100644
--- a/src/renderer/__mocks__/state-mocks.ts
+++ b/src/renderer/__mocks__/state-mocks.ts
@@ -43,7 +43,7 @@ const mockNotificationSettings: NotificationSettingsState = {
   showPills: true,
   showNumber: true,
   participating: false,
-  showReadNotifications: true,
+  showReadNotifications: false,
   markAsDoneOnOpen: false,
   markAsDoneOnUnsubscribe: false,
   delayNotificationState: false,
diff --git a/src/renderer/components/notifications/__snapshots__/NotificationRow.test.tsx.snap b/src/renderer/components/notifications/__snapshots__/NotificationRow.test.tsx.snap
index 8227203cc..59230905d 100644
--- a/src/renderer/components/notifications/__snapshots__/NotificationRow.test.tsx.snap
+++ b/src/renderer/components/notifications/__snapshots__/NotificationRow.test.tsx.snap
@@ -2721,35 +2721,6 @@ exports[`renderer/components/notifications/NotificationRow.tsx should render its
                   />
                 </svg>
               </button>
-              <button
-                aria-label="Mark as read"
-                class="prc-Button-ButtonBase-9n-Xk prc-Button-IconButton-fyge7"
-                data-component="IconButton"
-                data-loading="false"
-                data-no-visuals="true"
-                data-size="small"
-                data-testid="notification-mark-as-read"
-                data-variant="invisible"
-                title="Mark as read"
-                type="button"
-              >
-                <svg
-                  aria-hidden="true"
-                  class="octicon octicon-read"
-                  display="inline-block"
-                  fill="currentColor"
-                  focusable="false"
-                  height="16"
-                  overflow="visible"
-                  style="vertical-align: text-bottom;"
-                  viewBox="0 0 16 16"
-                  width="16"
-                >
-                  <path
-                    d="M7.115.65a1.752 1.752 0 0 1 1.77 0l6.25 3.663c.536.314.865.889.865 1.51v6.427A1.75 1.75 0 0 1 14.25 14H1.75A1.75 1.75 0 0 1 0 12.25V5.823c0-.621.33-1.196.865-1.51Zm1.011 1.293a.252.252 0 0 0-.252 0l-5.72 3.353L6.468 7.76a2.748 2.748 0 0 1 3.066 0l4.312-2.464-5.719-3.353ZM13.15 12.5 8.772 9.06a1.25 1.25 0 0 0-1.544 0L2.85 12.5Zm1.35-5.85-3.687 2.106 3.687 2.897ZM5.187 8.756 1.5 6.65v5.003Z"
-                  />
-                </svg>
-              </button>
               <button
                 aria-label="Unsubscribe from thread"
                 class="prc-Button-ButtonBase-9n-Xk prc-Button-IconButton-fyge7"
@@ -3079,35 +3050,6 @@ exports[`renderer/components/notifications/NotificationRow.tsx should render its
                 />
               </svg>
             </button>
-            <button
-              aria-label="Mark as read"
-              class="prc-Button-ButtonBase-9n-Xk prc-Button-IconButton-fyge7"
-              data-component="IconButton"
-              data-loading="false"
-              data-no-visuals="true"
-              data-size="small"
-              data-testid="notification-mark-as-read"
-              data-variant="invisible"
-              title="Mark as read"
-              type="button"
-            >
-              <svg
-                aria-hidden="true"
-                class="octicon octicon-read"
-                display="inline-block"
-                fill="currentColor"
-                focusable="false"
-                height="16"
-                overflow="visible"
-                style="vertical-align: text-bottom;"
-                viewBox="0 0 16 16"
-                width="16"
-              >
-                <path
-                  d="M7.115.65a1.752 1.752 0 0 1 1.77 0l6.25 3.663c.536.314.865.889.865 1.51v6.427A1.75 1.75 0 0 1 14.25 14H1.75A1.75 1.75 0 0 1 0 12.25V5.823c0-.621.33-1.196.865-1.51Zm1.011 1.293a.252.252 0 0 0-.252 0l-5.72 3.353L6.468 7.76a2.748 2.748 0 0 1 3.066 0l4.312-2.464-5.719-3.353ZM13.15 12.5 8.772 9.06a1.25 1.25 0 0 0-1.544 0L2.85 12.5Zm1.35-5.85-3.687 2.106 3.687 2.897ZM5.187 8.756 1.5 6.65v5.003Z"
-                />
-              </svg>
-            </button>
             <button
               aria-label="Unsubscribe from thread"
               class="prc-Button-ButtonBase-9n-Xk prc-Button-IconButton-fyge7"
diff --git a/src/renderer/components/settings/NotificationSettings.test.tsx b/src/renderer/components/settings/NotificationSettings.test.tsx
index 3fc3650a3..c50c65f9d 100644
--- a/src/renderer/components/settings/NotificationSettings.test.tsx
+++ b/src/renderer/components/settings/NotificationSettings.test.tsx
@@ -283,7 +283,7 @@ describe('renderer/components/settings/NotificationSettings.tsx', () => {
     expect(updateSettingMock).toHaveBeenCalledTimes(1);
     expect(updateSettingMock).toHaveBeenCalledWith(
       'showReadNotifications',
-      false,
+      true,
     );
   });
 
diff --git a/src/renderer/components/settings/NotificationSettings.tsx b/src/renderer/components/settings/NotificationSettings.tsx
index 7e1c170c1..a042e08fd 100644
--- a/src/renderer/components/settings/NotificationSettings.tsx
+++ b/src/renderer/components/settings/NotificationSettings.tsx
@@ -338,12 +338,16 @@ export const NotificationSettings: FC = () => {
           tooltip={
             <Stack direction="vertical" gap="condensed">
               <Text>
-                When <Text as="u">checked</Text>, {APPLICATION.NAME} will
-                display both read and unread notifications.
+                When <Text as="u">checked</Text>, {APPLICATION.NAME} will fetch
+                and display both read and unread notifications.
               </Text>
               <Text>
                 When <Text as="u">unchecked</Text>, {APPLICATION.NAME} will only
-                display unread notifications.
+                fetch and display unread notifications.
+              </Text>
+              <Text className="text-gitify-caution">
+                ⚠️ Enabling this setting will increase API usage and may cause
+                rate limiting for users with many notifications.
               </Text>
             </Stack>
           }
diff --git a/src/renderer/context/defaults.ts b/src/renderer/context/defaults.ts
index 5ab2ae413..6b615a99d 100644
--- a/src/renderer/context/defaults.ts
+++ b/src/renderer/context/defaults.ts
@@ -36,7 +36,7 @@ const defaultNotificationSettings: NotificationSettingsState = {
   showPills: true,
   showNumber: true,
   participating: false,
-  showReadNotifications: true,
+  showReadNotifications: false,
   markAsDoneOnOpen: false,
   markAsDoneOnUnsubscribe: false,
   delayNotificationState: false,
diff --git a/src/renderer/utils/api/client.test.ts b/src/renderer/utils/api/client.test.ts
index 0b16d0467..967a5f43c 100644
--- a/src/renderer/utils/api/client.test.ts
+++ b/src/renderer/utils/api/client.test.ts
@@ -75,6 +75,44 @@ describe('renderer/utils/api/client.ts', () => {
         data: {},
       });
     });
+
+    it('should include all=true when showReadNotifications is enabled', async () => {
+      const mockSettings: Partial<SettingsState> = {
+        participating: false,
+        showReadNotifications: true,
+      };
+
+      await listNotificationsForAuthenticatedUser(
+        mockGitHubCloudAccount,
+        mockSettings as SettingsState,
+      );
+
+      expect(axios).toHaveBeenCalledWith({
+        url: 'https://api.github.com/notifications?participating=false&all=true',
+        headers: mockNonCachedAuthHeaders,
+        method: 'GET',
+        data: {},
+      });
+    });
+
+    it('should not include all parameter when showReadNotifications is disabled', async () => {
+      const mockSettings: Partial<SettingsState> = {
+        participating: false,
+        showReadNotifications: false,
+      };
+
+      await listNotificationsForAuthenticatedUser(
+        mockGitHubCloudAccount,
+        mockSettings as SettingsState,
+      );
+
+      expect(axios).toHaveBeenCalledWith({
+        url: 'https://api.github.com/notifications?participating=false',
+        headers: mockNonCachedAuthHeaders,
+        method: 'GET',
+        data: {},
+      });
+    });
   });
 
   it('markNotificationThreadAsRead - should mark notification thread as read', async () => {
diff --git a/src/renderer/utils/api/client.ts b/src/renderer/utils/api/client.ts
index 7806dc75b..be43454cc 100644
--- a/src/renderer/utils/api/client.ts
+++ b/src/renderer/utils/api/client.ts
@@ -67,6 +67,10 @@ export function listNotificationsForAuthenticatedUser(
   url.pathname += 'notifications';
   url.searchParams.append('participating', String(settings.participating));
 
+  if (settings.showReadNotifications) {
+    url.searchParams.append('all', 'true');
+  }
+
   return apiRequestAuth(
     url.toString() as Link,
     'GET',
diff --git a/src/renderer/utils/notifications/filters/filter.test.ts b/src/renderer/utils/notifications/filters/filter.test.ts
index 139860547..e9ff59d33 100644
--- a/src/renderer/utils/notifications/filters/filter.test.ts
+++ b/src/renderer/utils/notifications/filters/filter.test.ts
@@ -96,6 +96,7 @@ describe('renderer/utils/notifications/filters/filter.ts', () => {
         mockNotifications[1].subject.type = 'PullRequest';
         const result = filterBaseNotifications(mockNotifications, {
           ...mockSettings,
+          showReadNotifications: true,
           filterSubjectTypes: ['Issue'],
         });
 
@@ -108,6 +109,7 @@ describe('renderer/utils/notifications/filters/filter.ts', () => {
         mockNotifications[1].reason = 'manual';
         const result = filterBaseNotifications(mockNotifications, {
           ...mockSettings,
+          showReadNotifications: true,
           filterReasons: ['manual'],
         });
 
@@ -118,6 +120,7 @@ describe('renderer/utils/notifications/filters/filter.ts', () => {
       it('should filter notifications that match include organization', async () => {
         const result = filterBaseNotifications(mockNotifications, {
           ...mockSettings,
+          showReadNotifications: true,
           filterIncludeSearchTokens: ['org:gitify-app' as SearchToken],
         });
 
@@ -128,6 +131,7 @@ describe('renderer/utils/notifications/filters/filter.ts', () => {
       it('should filter notifications that match exclude organization', async () => {
         const result = filterBaseNotifications(mockNotifications, {
           ...mockSettings,
+          showReadNotifications: true,
           filterExcludeSearchTokens: ['org:github' as SearchToken],
         });
 
@@ -138,6 +142,7 @@ describe('renderer/utils/notifications/filters/filter.ts', () => {
       it('should filter notifications that match include repository', async () => {
         const result = filterBaseNotifications(mockNotifications, {
           ...mockSettings,
+          showReadNotifications: true,
           filterIncludeSearchTokens: ['repo:gitify-app/gitify' as SearchToken],
         });
 
@@ -148,6 +153,7 @@ describe('renderer/utils/notifications/filters/filter.ts', () => {
       it('should filter notifications that match exclude repository', async () => {
         const result = filterBaseNotifications(mockNotifications, {
           ...mockSettings,
+          showReadNotifications: true,
           filterExcludeSearchTokens: ['repo:github/github' as SearchToken],
         });