Changeset 6e0249f in Klonkt for src/routes

Timestamp:

06/14/2026 03:21:25 PM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

4c5169f

Parents:

743bb81

Message:

hub: generic home page (not owned by any user) + admin-managed branding

The hub home page was the oldest site (a user's PrutFolio). Now it is a
generic page: title/tagline/intro come from global settings (hub_title/
hub_tagline/hub_intro), editable in Admin -> Settings (hub only). The roster
now shows ALL users (including the admin's own PrutFolio under /user/<slug>)
— no site is singled out as "the company" any more.

POST /admin/settings only updates tenancy when the field is present (a
separate branding form no longer accidentally flips the mode).

Co-Authored-By: Claude <noreply@…>

Location:

src/routes

Files:

• admin-settings.js (modified) (4 diffs)
• hub.js (modified) (2 diffs)

src/routes/admin-settings.js

@@ -1 +1 @@
 /**
- * Admin: globale instellingen — nu de tenancy-modus (Solo/Hub).
+ * Admin: globale instellingen.
+ *  - tenancy-modus (Solo/Hub)
+ *  - hub-branding (naam/tagline/intro van de generieke hub-hoofdpagina)
  *
- * GET  /admin/settings        -> toon huidige modus + uitleg
- * POST /admin/settings        -> sla modus op (god-only)
+ * GET  /admin/settings   -> toon huidige instellingen
+ * POST /admin/settings   -> sla op (god-only)
  *
- * Schakelen is niet-destructief: Solo verbergt alleen de multi-site-onderdelen
- * en routeert naar de primaire site; er wordt niets verwijderd.
+ * De hub-pagina is generiek (van geen enkele user); deze branding leeft in
+ * globale settings, niet in een site.
  */
 
@@ -12 +14 @@
 import { renderPage } from '../middleware/render.js';
 import { requireGod } from '../middleware/auth.js';
-import { getTenancy, setTenancy } from '../services/SettingsService.js';
+import { getTenancy, setTenancy, getSetting, setSetting } from '../services/SettingsService.js';
 
 const router = express.Router();
@@ -21 +23 @@
     bodyClass: 'on-admin',
     tenancy: getTenancy(),
+    hubTitle: getSetting('hub_title') || '',
+    hubTagline: getSetting('hub_tagline') || '',
+    hubIntro: getSetting('hub_intro') || '',
     success: req.query.success || null,
   });
@@ -26 +31 @@
 
 router.post('/', requireGod, (req, res) => {
-  setTenancy(req.body.tenancy === 'hub' ? 'hub' : 'solo');
-  res.redirect('/admin/settings?success=' + encodeURIComponent('Modus opgeslagen'));
+  if (typeof req.body.tenancy !== 'undefined') {
+    setTenancy(req.body.tenancy === 'hub' ? 'hub' : 'solo');
+  }
+  if (typeof req.body.hub_title !== 'undefined') {
+    setSetting('hub_title', (req.body.hub_title || '').toString().slice(0, 80).trim());
+  }
+  if (typeof req.body.hub_tagline !== 'undefined') {
+    setSetting('hub_tagline', (req.body.hub_tagline || '').toString().slice(0, 120).trim());
+  }
+  if (typeof req.body.hub_intro !== 'undefined') {
+    setSetting('hub_intro', (req.body.hub_intro || '').toString().slice(0, 400).trim());
+  }
+  res.redirect('/admin/settings?success=' + encodeURIComponent('Opgeslagen'));
 });
 

src/routes/hub.js

@@ -10 +10 @@
 import db from '../config/database.js';
 import { renderPage } from '../middleware/render.js';
-import { getTenancy } from '../services/SettingsService.js';
+import { getTenancy, getSetting } from '../services/SettingsService.js';
 
 const router = express.Router();
@@ -35 +35 @@
   `).all();
 
-  // De PrutFolio's (sites) voor de gebruikers-lijst.
-  const sites = db.prepare(`
-    SELECT s.slug, s.title, s.profile_photo,
-           u.username AS owner_username,
+  // ALLE PrutFolio's (elke user, incl. de admin z'n eigen site) voor de roster.
+  const artists = db.prepare(`
+    SELECT s.slug, s.title, s.tagline, s.profile_photo,
            (SELECT COUNT(*) FROM posts WHERE site_id = s.id AND status = 'published') AS post_count
     FROM sites s
-    LEFT JOIN users u ON u.id = s.owner_id
     ORDER BY s.created_at ASC
   `).all();
 
-  // De primaire/oudste site geldt als de bedrijfssite (label). De roster toont
-  // alleen de artiesten (alle overige sites), niet de bedrijfssite zelf.
-  const company = res.locals.site || null;
-  const artists = company ? sites.filter((s) => s.slug !== company.slug) : sites;
+  // De hub-pagina is GENERIEK (van geen enkele user) — branding komt uit globale
+  // instellingen die de admin in Beheer beheert, niet uit een site.
+  const hub = {
+    title: getSetting('hub_title') || 'Overzicht',
+    tagline: getSetting('hub_tagline') || '',
+    intro: getSetting('hub_intro') || '',
+  };
 
   renderPage(req, res, 'pages/hub-home', {
-    pageTitle: company ? company.title : 'Overzicht',
-    socialDescr: (company && (company.description || company.tagline)) || '',
+    pageTitle: hub.title,
+    socialDescr: hub.intro || hub.tagline || '',
     bodyClass: 'on-home on-hub',
-    company,
+    hub,
     artists,
     posts,