Changes in / [f549cf5:09fc5fb] in Klonkt

Files:

• src/routes/feed.js (modified) (4 diffs)
• src/services/guardianship/queues.js (modified) (1 diff)
• test/feed-tracks-only.test.js (deleted)
• test/help-c2s.test.js (modified) (2 diffs)

src/routes/feed.js

@@ -44 +44 @@
 
 // ==================== RSS 2.0 ====================
-/**
- * Twee feeds, een bouwer.
- *
- *   /feed.xml    de site: posts EN open tracks door elkaar, chronologisch
- *   /tracks.xml  alleen de open tracks -- de hele site als muziekkanaal
- *
- * WAAROM DIE TWEEDE (Robins vraag, 10-8). Hij vroeg naar een "hele site, open
- * tracks"-library voor Funkwhale. Als AS2-object helpt zo'n ding daar niet: een
- * Funkwhale-KANAAL wijst nergens naar een library, en hun library is een eigen
- * actor met inbox en sleutel -- de optie die op 7 augustus is afgewezen. Maar de
- * vorm waarin Funkwhale een kanaal WEL uitgeeft is RSS met iTunes en een
- * enclosure per item, en dat is precies wat Klonkt hier al doet. Naast een echte
- * kanaalfeed gelegd (audio.pepemoss.com/api/v1/channels/tnd/rss): dezelfde
- * namespaces, dezelfde enclosure.
- *
- * Het enige verschil was dat onze feed ook gewone posts draagt. Een muziekkanaal
- * met blogberichten ertussen is er geen, dus die feed staat er nu apart -- een
- * URL om te plakken, zonder dat er iets naar buiten geduwd wordt.
- */
-function stuurFeed(req, res, { alleenTracks }) {
+router.get('/feed.xml', (req, res) => {
   const site = res.locals.site;
   if (!site) return res.status(404).send('No site');
@@ -69 +50 @@
   const origin = siteOrigin(req);
   const base = origin + (res.locals.siteUrlBase || '');
-  const posts = alleenTracks ? [] : postsForFeed(site.id);
+  const posts = postsForFeed(site.id);
 
   // De tracks die deze site aan de federatie heeft opengezet, elk als eigen
@@ -84 +65 @@
   // poortregel en die hoort op een plek te staan.
   const tracks = siteOpenTracks(site.id);
-  const lastBuild = posts[0]?.published_at || tracks[0]?.created_at || new Date().toISOString();
+  const lastBuild = posts[0]?.published_at || new Date().toISOString();
 
   // De itunes-velden waar een podcast-app een kanaal aan herkent. Funkwhale
@@ -136 +117 @@
 <rss version="2.0" xmlns:atom="http://www.w3.org/2005/Atom" xmlns:content="http://purl.org/rss/1.0/modules/content/" xmlns:itunes="http://www.itunes.com/dtds/podcast-1.0.dtd">
   <channel>
-    <title>${escapeXml(alleenTracks ? `${site.title} \u2014 muziek` : site.title)}</title>
+    <title>${escapeXml(site.title)}</title>
     <link>${escapeXml(base + '/')}</link>
     <description>${escapeXml(site.description || site.tagline || '')}</description>
     <language>${escapeXml(site.language || 'nl')}</language>
     <lastBuildDate>${new Date(lastBuild).toUTCString()}</lastBuildDate>
-    <atom:link href="${escapeXml(base + (alleenTracks ? '/tracks.xml' : '/feed.xml'))}" rel="self" type="application/rss+xml" />
+    <atom:link href="${escapeXml(base + '/feed.xml')}" rel="self" type="application/rss+xml" />
 ${kanaalTags}
 ${items}
   </channel>
 </rss>`);
-}
-
-router.get('/feed.xml', (req, res) => stuurFeed(req, res, { alleenTracks: false }));
-// De hele site als muziekkanaal: alleen wat op de federatie openstaat.
-router.get('/tracks.xml', (req, res) => stuurFeed(req, res, { alleenTracks: true }));
+});
 
 // ==================== Atom 1.0 ====================

src/services/guardianship/queues.js

@@ -294 +294 @@
     'shaer:handledAt': h.state.handled ? h.state.handled.at : undefined,
     'shaer:pickedUpBy': h.state.pickedUpBy.map((p) => p.handle || p.uri),
-    // HOE OUD het oudste oppakken is (shaer-lgo). Barts besluit was dat
-    // "opgepikt" niet vervalt maar zichtbaar VEROUDERT -- het verschil tussen
-    // "er is iemand mee bezig" en "er was ooit iemand mee bezig". Het paneel
-    // toonde dat al; de apps konden het niet, want dit veld bleef hier liggen.
-    // Een oppak van vijf minuten zag er daar uit als een van vijf dagen, en de
-    // faalstand is hier nou juist "iedereen denkt dat het geregeld is".
-    //
-    // Als TIJDSTIP en niet als leeftijd: een leeftijd maakt elk antwoord anders
-    // en dan kan de ETag nooit gelijk zijn. De client rekent zelf terug, precies
-    // zoals guardian.js het doet.
-    'shaer:oldestPickupAt': h.state.oldestPickupAt || undefined,
     'shaer:formerWard': h.state.formerWard || undefined,
   }));

test/help-c2s.test.js

@@ -60 +60 @@
 });
 
-test('en de apps krijgen te horen HOE OUD dat oppakken is', () => {
-  // Barts besluit: opgepikt vervalt niet maar veroudert zichtbaar -- het
-  // verschil tussen "er is iemand mee bezig" en "er was ooit iemand mee bezig".
-  // Het paneel toonde dat al uit dezelfde staat; de apps kregen het veld niet,
-  // dus daar zag een oppak van vijf minuten eruit als een van vijf dagen.
-  const coll = queues.helpCollection('https://x/queues/help', 'oma');
-  const item = coll.orderedItems.find((i) => i.id === BOEI);
-  assert.ok(item['shaer:oldestPickupAt'], 'het veld staat er');
-  assert.ok(!Number.isNaN(Date.parse(item['shaer:oldestPickupAt'])), 'en is een leesbaar tijdstip');
-  // Een TIJDSTIP en geen leeftijd: een leeftijd maakt elk antwoord anders en
-  // dan kan de ETag nooit gelijk zijn. De client rekent zelf terug.
-  assert.equal(typeof item['shaer:oldestPickupAt'], 'string');
-});
-
 test('afsluiten vanuit de app sluit hem ook echt', async () => {
   await AP.ingestOutboxActivity(site(), user, marker('handled'));
@@ -89 +75 @@
   assert.ok(item['shaer:handledBy']);
   assert.equal(item['shaer:helpRequest'], true);
-  // En de ouderdom is WEG zodra hij is afgehandeld: hoe lang er iemand naar
-  // keek is dan geen informatie meer, en zou als "er wacht nog iets" lezen.
-  assert.equal(item['shaer:oldestPickupAt'], undefined);
 });