From 78669363b13c9a4c93bb7ffcc0fc82984c788f53 Mon Sep 17 00:00:00 2001 From: Nikhil <118773738+pablohashescobar@users.noreply.github.com> Date: Thu, 20 Jul 2023 18:53:02 +0530 Subject: [PATCH 1/2] fix: restrict notifications for created by when the actor is same (#1596) --- apiserver/plane/bgtasks/issue_activites_task.py | 1 + 1 file changed, 1 insertion(+) diff --git a/apiserver/plane/bgtasks/issue_activites_task.py b/apiserver/plane/bgtasks/issue_activites_task.py index 523e4bc30..105a05b56 100644 --- a/apiserver/plane/bgtasks/issue_activites_task.py +++ b/apiserver/plane/bgtasks/issue_activites_task.py @@ -1129,6 +1129,7 @@ def issue_activity( issue is not None and issue.created_by_id is not None and not issue.created_by.is_bot + and str(issue.created_by_id) != str(actor_id) ): issue_subscribers = issue_subscribers + [issue.created_by_id] From 546aa40aa3e7ba79fda9486c4e96d4d247baa5ee Mon Sep 17 00:00:00 2001 From: Dakshesh Jain <65905942+dakshesh14@users.noreply.github.com> Date: Thu, 20 Jul 2023 19:15:13 +0530 Subject: [PATCH 2/2] fix: un-read count not mutating (#1598) * refactor: height of popover & api fetch call * fix: notification subscribe endpoint * fix: un-read count not mutating --- apps/app/hooks/use-user-notifications.tsx | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/apps/app/hooks/use-user-notifications.tsx b/apps/app/hooks/use-user-notifications.tsx index 394628aee..1a4d6d8a8 100644 --- a/apps/app/hooks/use-user-notifications.tsx +++ b/apps/app/hooks/use-user-notifications.tsx @@ -74,6 +74,7 @@ const useUserNotification = () => { }) .finally(() => { notificationsMutate(); + mutateNotificationCount(); }); } else { notificationsMutate( @@ -96,6 +97,7 @@ const useUserNotification = () => { }) .finally(() => { notificationsMutate(); + mutateNotificationCount(); }); } }; @@ -114,6 +116,7 @@ const useUserNotification = () => { }) .finally(() => { notificationsMutate(); + mutateNotificationCount(); }); } else { notificationsMutate( @@ -127,6 +130,7 @@ const useUserNotification = () => { }) .finally(() => { notificationsMutate(); + mutateNotificationCount(); }); } }; @@ -145,6 +149,7 @@ const useUserNotification = () => { }) .finally(() => { notificationsMutate(); + mutateNotificationCount(); }); } else { notificationsMutate( @@ -160,6 +165,7 @@ const useUserNotification = () => { }) .finally(() => { notificationsMutate(); + mutateNotificationCount(); }); } };