Merge branch...

Merge branch '867-unreadnotificationscount-is-queried-when-accessing-notifications-even-though-there-is-no-need' into 'master'

Resolve "unreadNotificationsCount is queried when accessing notifications even though there is no need for it"

Closes #867

See merge request !1255

CHANGELOG.rst

@@ -14,6 +14,7 @@ Fixed
 
 * Progress page didn't work properly.
 * About page failed to load for apps with an unknown licence.
+* Querying for notification caused unnecessary database requests.
 * Loading bar didn't disappear on some pages after loading was finished.
 
 `3.1`_ - 2022-05-30

aleksis/core/frontend/components/notifications/NotificationList.vue

@@ -20,7 +20,7 @@
           v-if="
             myNotifications &&
             myNotifications.person &&
-            myNotifications.person.unreadNotificationsCount > 0
+            myNotifications.person.notifications.length > 0
           "
         >
           mdi-bell-badge-outline
@@ -38,7 +38,7 @@
         v-if="
           myNotifications.person &&
           myNotifications.person.notifications &&
-          myNotifications.person.notifications.length
+          unreadNotifications.length
         "
       >
         <v-subheader>{{ $t("notifications.notifications") }}</v-subheader>
@@ -86,5 +86,10 @@ export default {
       pollInterval: 30000,
     },
   },
+  computed: {
+    unreadNotifications() {
+      return this.myNotifications.filter(n => !n.read);
+    },
+  },
 };
 </script>

aleksis/core/frontend/components/notifications/myNotifications.graphql

 {
   myNotifications: whoAmI {
     person {
-      unreadNotificationsCount
       notifications {
         id
         title