From 2c53f7ae7c3b9ff0f0c816378a1af20a461f001c Mon Sep 17 00:00:00 2001
From: Bruno Quaresma <bruno@coder.com>
Date: Tue, 25 Mar 2025 14:29:52 -0300
Subject: [PATCH] feat: mark notification as read when action is clicked
 (#17095)

When a user clicks in a notification action we can infer the
notification was read.
---
 .../notifications/NotificationsInbox/InboxItem.tsx       | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/site/src/modules/notifications/NotificationsInbox/InboxItem.tsx b/site/src/modules/notifications/NotificationsInbox/InboxItem.tsx
index e097a6e296..0f66f0b71d 100644
--- a/site/src/modules/notifications/NotificationsInbox/InboxItem.tsx
+++ b/site/src/modules/notifications/NotificationsInbox/InboxItem.tsx
@@ -40,7 +40,14 @@ export const InboxItem: FC<InboxItemProps> = ({
 					{notification.actions.map((action) => {
 						return (
 							<Button variant="outline" size="sm" key={action.label} asChild>
-								<RouterLink to={action.url}>{action.label}</RouterLink>
+								<RouterLink
+									to={action.url}
+									onClick={() => {
+										onMarkNotificationAsRead(notification.id);
+									}}
+								>
+									{action.label}
+								</RouterLink>
 							</Button>
 						);
 					})}