Changeset b109a29 in Klonkt for src/routes/posts.js

Timestamp:

07/16/2026 08:02:27 PM (8 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

010e073

Parents:

87d2787

git-author:

Robin <roboburr@…> (07/16/2026 07:56:26 PM)

git-committer:

Robin <roboburr@…> (07/16/2026 08:02:27 PM)

Message:

Feature: Connect page merges following + followers into one view

/connect combines who you follow (ap_following) and who follows you
(ap_followers) into a single list keyed by actor_uri, each with a direction
badge (following →, follower ←, mutual ↔) and, for accounts we deliver to, a
light 'last delivery' line. Failing/never-reached followers move into a
collapsed 'Unreachable' section at the bottom with the remove action, keeping
the main list clean. The two Following/Followers tabs collapse into one
'Connect' tab; the old routes redirect to /connect so links keep working.
i18n NL/EN/DE. Verified locally: merge/direction/unreachable logic + render.

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

File:

• src/routes/posts.js (modified) (1 diff)

src/routes/posts.js

@@ -827 +827 @@
 
 // Volgend — manage the accounts you follow (+ per-account auto-boost toggles).
-router.get('/following', requireSiteManager, (req, res) => {
-  const site = res.locals.site;
-  const following = site ? ActivityPubService.listFollowing(site.slug) : [];
-  renderPage(req, res, 'pages/following', {
-    pageTitle: 'Volgend', bodyClass: 'on-special',
-    following,
+// Connect = who you follow + who follows you, merged into one page with direction
+// (following →, follower ←, mutual ↔) and per-account delivery health. Replaces the
+// separate Following/Followers pages, which redirect here so old links keep working.
+router.get('/connect', requireSiteManager, (req, res) => {
+  const site = res.locals.site;
+  const connections = site ? ActivityPubService.listConnections(site.slug) : [];
+  renderPage(req, res, 'pages/connect', {
+    pageTitle: 'Connect', bodyClass: 'on-special',
+    connections,
     success: req.query.success || null, error: req.query.error || null,
   });
 });
-
-// Followers (remote AP actors following us) + per-account delivery health, so dead
-// accounts (never/last-delivered long ago) can be pruned after a manual check.
-router.get('/followers', requireSiteManager, (req, res) => {
-  const site = res.locals.site;
-  const followers = site ? ActivityPubService.listFollowers(site.slug) : [];
-  renderPage(req, res, 'pages/followers', {
-    pageTitle: 'Volgers', bodyClass: 'on-special',
-    followers,
-    success: req.query.success || null, error: req.query.error || null,
-  });
-});
+router.get('/following', requireSiteManager, (req, res) => res.redirect(`${res.locals.siteUrlBase || ''}/connect`));
+router.get('/followers', requireSiteManager, (req, res) => res.redirect(`${res.locals.siteUrlBase || ''}/connect`));
 
 router.post('/followers/:id/remove', requireSiteManager, (req, res) => {
   const site = res.locals.site;
   const base = res.locals.siteUrlBase || '';
-  if (!site) return res.redirect(`${base}/followers`);
+  if (!site) return res.redirect(`${base}/connect`);
   const ok = ActivityPubService.removeFollower(site.slug, parseInt(req.params.id, 10) || 0);
-  return res.redirect(`${base}/followers?` + (ok
+  return res.redirect(`${base}/connect?` + (ok
     ? 'success=' + encodeURIComponent('Volger verwijderd')
     : 'error=' + encodeURIComponent('Volger niet gevonden')));