open.source/manifold
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove query from notif isSeen logic

Browse Source
This commit is contained in:
Ian Philips 2022-07-18 08:18:40 -06:00
parent d012561c50
commit eb4906cb97
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
web/components/nav/sidebar.tsx
View File

@ -289,15 +289,17 @@ function GroupsList(props: {
// Set notification as seen if our current page is equal to the isSeenOnHref property // Set notification as seen if our current page is equal to the isSeenOnHref property
useEffect(() => { useEffect(() => {
const currentPageGroupSlug = currentPage.split('/')[2] const currentPageWithoutQuery = currentPage.split('?')[0]
const currentPageGroupSlug = currentPageWithoutQuery.split('/')[2]
preferredNotifications.forEach((notification) => { preferredNotifications.forEach((notification) => {
if ( if (
notification.isSeenOnHref === currentPage || notification.isSeenOnHref === currentPage ||
// Old chat style group chat notif ended just with the group slug // Old chat style group chat notif was just /group/slug
notification.isSeenOnHref?.includes(currentPageGroupSlug) || (notification.isSeenOnHref &&
currentPageWithoutQuery.includes(notification.isSeenOnHref)) ||
// They're on the home page, so if they've a chat notif, they're seeing the chat // They're on the home page, so if they've a chat notif, they're seeing the chat
(notification.isSeenOnHref?.endsWith(GROUP_CHAT_SLUG) && (notification.isSeenOnHref?.endsWith(GROUP_CHAT_SLUG) &&
currentPage.endsWith(currentPageGroupSlug)) currentPageWithoutQuery.endsWith(currentPageGroupSlug))
) { ) {
setNotificationsAsSeen([notification]) setNotificationsAsSeen([notification])
} }