From 3d06ceef4d6949e52894a67415f92a2ef3c8c273 Mon Sep 17 00:00:00 2001 From: Riyadh Date: Sun, 17 May 2026 15:42:29 +0000 Subject: [PATCH] Show the notifications app on the main page again Remove hardcoded exclusion for the notifications app in the home page component. --- artifacts/tx-os/src/pages/home.tsx | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/artifacts/tx-os/src/pages/home.tsx b/artifacts/tx-os/src/pages/home.tsx index 9a98d9c4..247b525f 100644 --- a/artifacts/tx-os/src/pages/home.tsx +++ b/artifacts/tx-os/src/pages/home.tsx @@ -362,19 +362,7 @@ export default function HomePage() { useEffect(() => { if (!apps) return; - // #571: hide the Notifications app from the home grid (and the - // bottom dock — see `dockApps` filter below) regardless of its - // `isActive` flag. The top-bar bell icon is the canonical entry - // point for notifications, so a tile in the launcher is - // redundant. We filter at the client (rather than disabling the - // app server-side) so existing environments where Notifications - // is enabled stay enabled at the data layer — keeps deep links - // / direct navigation to `/notifications` working, and lets - // admins flip the app's visibility back without a DB change if - // they ever want a tile back. - const activeApps = apps.filter( - (a) => a.isActive && a.slug !== "notifications", - ); + const activeApps = apps.filter((a) => a.isActive); setOrderedApps((prev) => { if (!prev) return activeApps; const byId = new Map(activeApps.map((a) => [a.id, a]));