Show the notifications app on the main page again
Remove hardcoded exclusion for the notifications app in the home page component.
This commit is contained in:
@@ -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]));
|
||||
|
||||
Reference in New Issue
Block a user