fix: Set announcement as read when opened

Author: Ushie

src/routes/announcements/AnnouncementBanner.svelte

@@ -13,23 +13,34 @@
 
 	$: {
 		if ($query.data?.announcements && $query.data.announcements.length > 0) {
-			const announcement = $query.data.announcements[0];
-			if (!$read_announcements.has(announcement.id)) {
+			const nonArchived = $query.data.announcements.filter((a) => !a.archived_at);
+			const announcement = nonArchived[0];
+
+			if (announcement && !$read_announcements.has(announcement.id)) {
 				latestUnreadAnnouncement = announcement;
 			} else {
 				latestUnreadAnnouncement = undefined;
 			}
 		}
 	}
 
-	const handleClick = () =>
-		latestUnreadAnnouncement && goto(`/announcements/${latestUnreadAnnouncement.id}`);
+	function setAsRead() {
+		if (!latestUnreadAnnouncement) return;
+		$read_announcements.add(latestUnreadAnnouncement.id);
+		localStorage.setItem('read_announcements', JSON.stringify(Array.from($read_announcements)));
+		latestUnreadAnnouncement = undefined;
+	}
+
+	function handleClick() {
+		if (latestUnreadAnnouncement) {
+			goto(`/announcements/${latestUnreadAnnouncement.id}`);
+			setAsRead();
+		}
+	}
 
 	function handleClose() {
 		if (latestUnreadAnnouncement && browser) {
-			$read_announcements.add(latestUnreadAnnouncement.id);
-			localStorage.setItem('read_announcements', JSON.stringify(Array.from($read_announcements)));
-			latestUnreadAnnouncement = undefined;
+			setAsRead();
 		}
 	}