Changeset 98ecf51 in Klonkt for src/views/pages/admin-site-edit.ejs

Timestamp:

06/18/2026 03:19:09 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

7881080

Parents:

6a2abb1

Message:

Hub #1+#2: site admin permissions work + assign owner to another user

#1 — canAdminSite was reading user.siteRoles which was NEVER populated (dead
code) → only god and the literal owner passed admin checks. Now canAdminSite
queries site_members directly (an assigned co-admin). requireSiteManager/BySlug
use canAdminSite so collaborators can access site management too.

#2 — owner_id was hard-coded to the creating god → you couldn't assign a site to
another user (the core of hub mode was missing). Now: god-only "Owner" select in
the site form; create/save set owner_id + a site_members-admin row for that owner
(upsert). Non-god doesn't see the field and cannot change the owner.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@…>

File:

• src/views/pages/admin-site-edit.ejs (modified) (1 diff)

src/views/pages/admin-site-edit.ejs

@@ -24 +24 @@
         <input type="text" name="title" value="<%= site.title || '' %>" required maxlength="200">
       </label>
+      <%# Eigenaar (her)toewijzen — ALLEEN god. In hub-modus geef je zo elke
+          gebruiker z'n eigen, zelf te beheren Klonkt. Niet-god ziet dit niet. %>
+      <% if (user && user.role === 'god') { %>
+      <label>
+        <span>Eigenaar <small class="form-hint-inline">— wie deze Klonkt zelf mag beheren</small></span>
+        <select name="owner_id">
+          <% (typeof users !== 'undefined' ? users : []).forEach(function(u){ %>
+            <option value="<%= u.id %>" <%= (site.owner_id === u.id) ? 'selected' : '' %>><%= u.username %><%= u.role === 'god' ? ' (god)' : '' %></option>
+          <% }); %>
+        </select>
+      </label>
+      <% } %>
       <label>
         <span>Tagline <small class="form-hint-inline">— korte oneliner</small></span>