Changeset 65c5ec6 in Klonkt for src/services/ActivityPubService.js

Timestamp:

08/10/2026 03:26:40 AM (4 weeks ago)

Author:

Bart <bart@…>

Branches:

main

Children:

8064da1

Parents:

6c83c9e

Message:

Een volgverzoek is geen adoptie-aanvraag

Drie meldingen deelden één tekstpaar, en dat paar hoort ergens anders: de
event-map koppelt n_guard_cog_* aan offer_for_ward, een echte mede-voogdij. De
volgpoort leende het. Een guardian kreeg dus "Mede-voogdij gevraagd -- er is een
guardian-aanvraag voor {who}" op zijn telefoon terwijl zijn ward alleen vroeg of
het iemand mocht volgen, met de verkeerde persoon als onderwerp. Dat meldt niet
te weinig, dat meldt iets anders.

Twee eigen paren nu, per richting, met allebei de namen erin: wie er vraagt en
om wie het gaat. Zonder die tweede naam blijft een melding over een kind een
melding waarin het kind niet voorkomt.

Bij de doorgestuurde review stond uitgaand al drie regels boven de push, en
werd daar weer weggegooid. Nu leest hij hem.

n_guard_cog_* blijft ongemoeid; die is van offer_for_ward en klopte daar altijd
al.

Co-Authored-By: Claude Opus 5 <claude@…>

File:

• src/services/ActivityPubService.js (modified) (3 diffs)

src/services/ActivityPubService.js

@@ -2273 +2273 @@
         if (isLocal) {
           const L = pushLang(gslug);
-          pushEvent(gslug, { type: 'guardian', title: i18nT(L, 'push.n_guard_cog_t'), body: i18nT(L, 'push.n_guard_cog_b', { who: fi.name || fi.handle || i18nT(L, 'notif.someone') }), url: `${pushPrefix(gslug)}/guardian` });
+          // Een volgverzoek is geen mede-voogdij. Deze push leende de tekst van
+          // offer_for_ward en meldde dus een adoptie die niet gebeurde -- met de
+          // volger als onderwerp. Eigen woorden, en allebei de namen erin: wie
+          // er vraagt, en om wie het gaat (shaer-p729).
+          pushEvent(gslug, { type: 'guardian', title: i18nT(L, 'push.n_guard_folin_t'), body: i18nT(L, 'push.n_guard_folin_b', { who: fi.name || fi.handle || i18nT(L, 'notif.someone'), ward: slug }), url: `${pushPrefix(gslug)}/guardian` });
         } else {
           fetchActor(g).then((ga) => {
@@ -5515 +5519 @@
     if (isLocal) {
       const L = pushLang(gslug);
-      pushEvent(gslug, { type: 'guardian', title: i18nT(L, 'push.n_guard_cog_t'), body: i18nT(L, 'push.n_guard_cog_b', { who: ti.name || ti.handle || i18nT(L, 'notif.someone') }), url: `${pushPrefix(gslug)}/guardian` });
+      // De andere richting, en dus andere woorden: hier vraagt het kind of het
+      // iemand mag volgen. Met dezelfde tekst als hierboven kon een guardian
+      // op zijn telefoon niet zien wie er nu eigenlijk om wie vroeg.
+      pushEvent(gslug, { type: 'guardian', title: i18nT(L, 'push.n_guard_folout_t'), body: i18nT(L, 'push.n_guard_folout_b', { who: ti.name || ti.handle || i18nT(L, 'notif.someone'), ward: slug }), url: `${pushPrefix(gslug)}/guardian` });
     } else {
       fetchActor(g).then((ga) => {
@@ -5625 +5632 @@
       });
       const L = pushLang(gslug);
-      pushEvent(gslug, { type: 'guardian', title: i18nT(L, 'push.n_guard_cog_t'), body: i18nT(L, 'push.n_guard_cog_b', { who: fai.name || fai.handle || i18nT(L, 'notif.someone') }), url: `${pushPrefix(gslug)}/guardian` });
+      // `uitgaand` staat hier al, drie regels hoger, en werd voor de melding
+      // weer weggegooid: elke richting kreeg dezelfde tekst, geleend van
+      // offer_for_ward. Op de telefoon las een volgverzoek dus als een
+      // adoptie-aanvraag, en beide richtingen als elkaar.
+      const wardNaam = (wardDoc && (wardDoc.preferredUsername || wardDoc.name)) || slugFromActorUrl(wardUri) || wardUri;
+      const anderNaam = uitgaand
+        ? ((dai && (dai.name || dai.handle)) || i18nT(L, 'notif.someone'))
+        : (fai.name || fai.handle || i18nT(L, 'notif.someone'));
+      pushEvent(gslug, {
+        type: 'guardian',
+        title: i18nT(L, uitgaand ? 'push.n_guard_folout_t' : 'push.n_guard_folin_t'),
+        body: i18nT(L, uitgaand ? 'push.n_guard_folout_b' : 'push.n_guard_folin_b', { who: anderNaam, ward: wardNaam }),
+        url: `${pushPrefix(gslug)}/guardian`,
+      });
       stored = true;
     }