Changeset 79d1281 in Klonkt

Timestamp:

06/26/2026 02:28:50 PM (2 months ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

fdd0d61

Parents:

c69267f

Message:

fix(fedi): /notifications showed old (tab-less) view — route collision after rename

The /meldingen -> /notifications rename collided with the pre-existing (dead) user-
notifications route mounted at /notifications, which won and rendered the old tab-less
view. Removed that dead route mount + import, so the fediverse /notifications route (in
postsRoutes, with the tab bar) handles it.

File:

• src/server.js (modified) (2 diffs)

src/server.js

@@ -26 +26 @@
 import authRoutes from './routes/auth.js';
 import accountRoutes from './routes/account.js';
-import notificationsRoutes from './routes/notifications.js';
 import adminRoutes from './routes/admin.js';
 import adminAudioRoutes from './routes/admin-audio.js';
@@ -286 +285 @@
 app.use('/auth', authRoutes);
 app.use('/account', accountRoutes);
-app.use('/notifications', notificationsRoutes);
+// NB: /notifications is the fediverse notifications page (in postsRoutes). The old
+// user-notifications route was removed — it collided with the fedi route after the
+// /meldingen -> /notifications rename, and the user-notifications system is dead.
 if (audioEnabled()) {
   app.use('/admin/audio', adminAudioRoutes);